View Issue Details

IDProjectCategoryView StatusLast Update
0029286LazarusDocumentationpublic2015-12-30 18:45
ReporterwpAssigned ToBart Broersma 
PrioritynormalSeverityminorReproducibilityalways
Status closedResolutionfixed 
Product Version1.7 (SVN)Product Build 
Target Version1.8Fixed in Version1.8 
Summary0029286: r51048 breaks compilation of html help
DescriptionIn r51048, documentation for the new html operations of the clipboard were added. The xml source file (clipboard.xml), however uses incorrect xml end tags (e.g. <short/> instead of </short>). In addition, an </element> tag is missing.

As a consequence, html help cannot be build any more using build_lcl_docs.

The provided patch fixes the issues.
TagsNo tags attached.
Fixed in Revisionr51085
LazTarget1.8
Widgetset
Attached Files
  • clipbrd.xml.patch (2,247 bytes)
    Index: docs/xml/lcl/clipbrd.xml
    ===================================================================
    --- docs/xml/lcl/clipbrd.xml	(revision 51054)
    +++ docs/xml/lcl/clipbrd.xml	(working copy)
    @@ -758,19 +758,20 @@
           </element>
           <!-- procedure Visibility: public -->
           <element name="TClipboard.SetAsHtml">
    -        <short>Puts a HTML string on the clipboard.<short/>
    +        <short>Puts a HTML string on the clipboard.</short>
             <descr>Puts a HTML string on the clipboard. On Windows it will automatically insert a
                    ClipBoard Header.
                    The procedure can also (optionally) set a provided plaintext representation
    -               of the HTLM on the ClipBoard.<descr/>
    +               of the HTLM on the ClipBoard.</descr>
             <errors/>
             <seealso/>
    +      </element>
           <!-- argument Visibility: default -->
           <element name="TClipboard.SetAsHtml.HTML">
    -        <short>The Html string that shall be put on the ClipBoard<short/>
    +        <short>The Html string that shall be put on the ClipBoard</short>
           </element>
           <element name="TClipboard.SetAsHtml.PlainText">
    -        <short>Optional plain text (string) representation of the HTML<short/>
    +        <short>Optional plain text (string) representation of the HTML</short>
             <descr>The optioal string paramater PlainText is meant to be a plain text representation
                    of the HTM string. The user is responsible for the contents of this variable.
                    The value of PlainText can later be retrieved via the
    @@ -780,10 +781,10 @@
           </element>
           <!-- function Visibility: public -->
           <element name="TClipboard.GetAsHtml">
    -        <short>Retrieves the HTML content of the clipboard (if available)<short/>
    +        <short>Retrieves the HTML content of the clipboard (if available)</short>
             <descr>Retrieves the HTML content of the clipboard (if available).
                    On Windows it will automatically remove the clipboard header that Windows inserts.
    -               The resulting HTML string will be UTF-8 encoded.<descr/>
    +               The resulting HTML string will be UTF-8 encoded.</descr>
             <errors/>
             <seealso/>
           </element>
    
    clipbrd.xml.patch (2,247 bytes)

Activities

wp

2015-12-30 04:23

developer  

clipbrd.xml.patch (2,247 bytes)
Index: docs/xml/lcl/clipbrd.xml
===================================================================
--- docs/xml/lcl/clipbrd.xml	(revision 51054)
+++ docs/xml/lcl/clipbrd.xml	(working copy)
@@ -758,19 +758,20 @@
       </element>
       <!-- procedure Visibility: public -->
       <element name="TClipboard.SetAsHtml">
-        <short>Puts a HTML string on the clipboard.<short/>
+        <short>Puts a HTML string on the clipboard.</short>
         <descr>Puts a HTML string on the clipboard. On Windows it will automatically insert a
                ClipBoard Header.
                The procedure can also (optionally) set a provided plaintext representation
-               of the HTLM on the ClipBoard.<descr/>
+               of the HTLM on the ClipBoard.</descr>
         <errors/>
         <seealso/>
+      </element>
       <!-- argument Visibility: default -->
       <element name="TClipboard.SetAsHtml.HTML">
-        <short>The Html string that shall be put on the ClipBoard<short/>
+        <short>The Html string that shall be put on the ClipBoard</short>
       </element>
       <element name="TClipboard.SetAsHtml.PlainText">
-        <short>Optional plain text (string) representation of the HTML<short/>
+        <short>Optional plain text (string) representation of the HTML</short>
         <descr>The optioal string paramater PlainText is meant to be a plain text representation
                of the HTM string. The user is responsible for the contents of this variable.
                The value of PlainText can later be retrieved via the
@@ -780,10 +781,10 @@
       </element>
       <!-- function Visibility: public -->
       <element name="TClipboard.GetAsHtml">
-        <short>Retrieves the HTML content of the clipboard (if available)<short/>
+        <short>Retrieves the HTML content of the clipboard (if available)</short>
         <descr>Retrieves the HTML content of the clipboard (if available).
                On Windows it will automatically remove the clipboard header that Windows inserts.
-               The resulting HTML string will be UTF-8 encoded.<descr/>
+               The resulting HTML string will be UTF-8 encoded.</descr>
         <errors/>
         <seealso/>
       </element>
clipbrd.xml.patch (2,247 bytes)

Bart Broersma

2015-12-30 17:55

developer   ~0088441

That's what you when doing it all manually.
Thanks for the patch.

wp

2015-12-30 18:45

developer   ~0088442

I know this too well...

build_lcl_docs working now. Thanks

Issue History

Date Modified Username Field Change
2015-12-30 04:23 wp New Issue
2015-12-30 04:23 wp Status new => assigned
2015-12-30 04:23 wp Assigned To => Bart Broersma
2015-12-30 04:23 wp File Added: clipbrd.xml.patch
2015-12-30 04:49 wp Description Updated View Revisions
2015-12-30 17:55 Bart Broersma Fixed in Revision => r51085
2015-12-30 17:55 Bart Broersma LazTarget - => 1.8
2015-12-30 17:55 Bart Broersma Note Added: 0088441
2015-12-30 17:55 Bart Broersma Status assigned => resolved
2015-12-30 17:55 Bart Broersma Fixed in Version => 1.8
2015-12-30 17:55 Bart Broersma Resolution open => fixed
2015-12-30 17:55 Bart Broersma Target Version => 1.8
2015-12-30 18:45 wp Note Added: 0088442
2015-12-30 18:45 wp Status resolved => closed