The following issues were noted in a roundtrip edition of a help page with the latest version of the HelpAuthoring.oxt extension. File analyses is (but not bounded to) helpcontent2/source/text/shared/optionen/01060500.xhp Issue 01: Keyword <filename> in <meta> looses initial slash (/) < <filename>/text/shared/optionen/01060500.xhp</filename> > <filename>text/shared/optionen/01060500.xhp</filename> Issue 02: Attribute l10n and oldref in <paragraph> deleted. Note: this may be irrelevant or deprecated. Attached the diff file
Issue 03: In another file(*) the paragraph attribute "localize" (=true/false) is deleted. This affects translations, and will pollute Pootle server with useless translations (e.g. translate keywords, numbers, etc...) (*) helpcontent2/source/text/scalc/01/statistics.xhp
Issue 04: In the same file as comment#1 the sections with id="howtoget" were renamed "SectionN", where N is a number >2. This is probably to keep section id's uniqueness inside the file. But we loose the "To access this command" sections, except the first occurrence. Manually naming id's as "howtoget" in the xhp file does work and the "To access this command..." sections are displayed. But I don't know if that obfuscate another issue.
(In reply to Olivier Hallot from comment #0) > Issue 02: > > Attribute l10n and oldref in <paragraph> deleted. > > Note: this may be irrelevant or deprecated. Read in table "Attributes" in section "Paragraphs" on page 58 in OOo2HelpAuthoring.pdf. The attribute 'oldref' can surely be removed totally. It is not required and the comment says "This contains the reference number used by the old help files and is only used for migration purposes." and migration is finished long ago. The attribute 'l10n' is described with "required" in that document, but when you look at the current /help/helpers/xmlhelp.dtd, it is actually not required. It has the same comment, so I think it can be removed too. Therefore the extension does it right, to remove both attributes. Perhaps these attributes should be removed in all help files.
(In reply to Olivier Hallot from comment #0) > Issue 01: > > Keyword <filename> in <meta> looses initial slash (/) > > < <filename>/text/shared/optionen/01060500.xhp</filename> > > > <filename>text/shared/optionen/01060500.xhp</filename> The problem is in Function SetMetaDataOnSave in module _Main. In string is build in line 57. The variable sDocRoot contains a character '/' as last character. Therefore subtracting sDocRoot removes the character '/' from the variable Path. I'm not sure about the solution: (A) in line 57 in Function SetMetaDataOnSave in module _Main - Path = Right(Path,Len(Path)-Len(sDocRoot)) + Path = Right(Path,Len(Path)-Len(sDocRoot)+1) or save the HelpPrefix without slash. I don't know what further implications that will have and where other parts would have to be adapted then. (B) in line 1057 in Function SetDocumentRoot in module Helpers - sHelpPrefix = oFolderDialog.getDirectory + "/" + sHelpPrefix = oFolderDialog.getDirectory I prefer (A). What do you think?
Reading comments it seems like this has been confirmed (as the discussion is about a solution not about the problem itself). Setting to NEW.
(In reply to Regina Henschel from comment #3) > (In reply to Olivier Hallot from comment #0) (....) > > Therefore the extension does it right, to remove both attributes. Perhaps > these attributes should be removed in all help files. Right, so since this is innocuous, let is stay as is for later cleanup.
(In reply to Regina Henschel from comment #4) > (A) in line 57 in Function SetMetaDataOnSave in module _Main > - Path = Right(Path,Len(Path)-Len(sDocRoot)) > + Path = Right(Path,Len(Path)-Len(sDocRoot)+1) Yes this was the same solution that i came up with and it works.
Let's split this bug to several ones, and have this as meta-issue for dependency :-)
(In reply to Olivier Hallot from comment #0) > The following issues were noted in a roundtrip edition of a help page with > the latest version of the HelpAuthoring.oxt extension. > > File analyses is (but not bounded to) > > helpcontent2/source/text/shared/optionen/01060500.xhp Which version number is this, as i'm unable to open that file with the latest version (3.1.0).
I've created bug 93979 for Issue 01, bug 93981 for Issue 03, and bug 93982 for Issue 04. I did not create a bug for Issue 02 as Regina's comment 3 says that these attributes are not necessary.
Got access to the dev-tools repo and have submitted 4 patches so far. Have title tag id attribute before xml-lang (merge directly to master) Embed tags should be on their own line (merge directly to master) Reorganize the menu according to usage https://gerrit.libreoffice.org/18408 Have level attribute of paragraph tag appear before xml-lang https://gerrit.libreoffice.org/18410 More patches coming soon and you can find the issues that i'm looking to patch as well as my patched version of the tool here - https://docs.google.com/document/d/1PWerIggak-XyuwtME-bYLtdnyw2LHoCUrJjhAqHcVO4/edit?usp=sharing' @Regina: My version should work with AOO as its based on 3.1.0. :D
Another set of patches sent into gerrit. Insert product name and version variables https://gerrit.libreoffice.org/18451 Add ability to reload the active help document https://gerrit.libreoffice.org/18452 Always jump to cursor position to insert new paragraph data https://gerrit.libreoffice.org/18454 Add wizard to creating new help file https://gerrit.libreoffice.org/18455 Dont clear help topic id and indexer https://gerrit.libreoffice.org/18456 Additional cleanup of the menu https://gerrit.libreoffice.org/18458
Another set of patches Dont set indexer during new help file wizard https://gerrit.libreoffice.org/18476 Open a embedded or linked help file and some menu cleanup https://gerrit.libreoffice.org/18485 Insert switch and switchinline tags https://gerrit.libreoffice.org/18486 I hope to push my last patches in today or tomorrow, depending on when these get pushed in, as there would be a merge conflict.
All the remaining patches are in and are just awaiting approval. Hopefully the new version can be published once they are in. I've been able to modify help files quite easily with the new version (the toolbar makes it alot easier) and my latest 2 help patches ( https://gerrit.libreoffice.org/18515, https://gerrit.libreoffice.org/18574 ) were primarily edited in the authoring tool, except for occurrences that trigger bug 94167 and when i just wanted to add a section to the beginning of help files. @Regina: Can you test bug 93982, basically olivier's issue number 4 listed on this page, as i wasnt able to reproduce this when editing with the authoring tool.
Helpauthoring extension is deprecated. closing wontfix.