Issue Details (XML | Word | Printable)

Key: FOR-546
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Unassigned
Reporter: Ron Blaschke
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Forrest

Sitemap reference doc should be updated to reflect plugin architecture

Created: 18/Jun/05 08:27 PM   Updated: 01/Mar/07 02:53 AM
Return to search
Component/s: Documentation and website
Affects Version/s: 0.7
Fix Version/s: 0.8

Time Tracking:
Not Specified


 Description  « Hide
The example at http://forrest.apache.org/0.7/docs/sitemap-ref.html#pdf uses a fo2pdf serializer. This probaly worked in versions < 0.7, because the PDF output feature was in the core, and the serializer was very likely declared there.

With version 0.7, PDF output is moved to a plugin, the fo2pdf serializer seems to be no longer there. The serializer should probably be added to the example, for completeness.

  <map:components>
    <map:serializers>
      <map:serializer name="fo2pdf"
                      src="org.apache.cocoon.serialization.FOPSerializer"
                      mime-type="application/pdf"/>
    </map:serializers>
  </map:components>

I found out about this because my sitemap uses the fo2pdf too (docbook to PDF), and had to add the serializer.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Ross Gardler added a comment - 18/Jun/05 10:29 PM
If you want to output PDF documents then just add the PDF plugin to your forrest.properties and request the PDF document as normal. There is no need to add anything to your sitemap.

The document you reference is intended to help people understand how the forrest sitemaps work so that they can learn how to extend them.

However, with the creation of plugins this document is largely outdated. It should be updated to reflect the use of, and creation of, plugins.

Ross Gardler added a comment - 18/Jun/05 10:31 PM
Updated priority and title to better reflect the issue.

Ross Gardler made changes - 18/Jun/05 10:31 PM
Field Original Value New Value
Summary fo2pdf serializer should be added to example Sitemap reference doc should be updated to reflect plugin architecture
Priority Trivial [ 5 ] Major [ 3 ]
Ron Blaschke added a comment - 18/Jun/05 10:50 PM
Sorry, should have been more specific. I am using the docbook stylesheets (http://docbook.sourceforge.net/projects/xsl/index.html), and my own pipeline.

    <map:pipeline>
      <map:match pattern="design.pdf">
        <map:generate src="{project:content.xdocs}design.xml"/>
        <map:transform src="resources/stylesheets/docbook/fo/docbook.xsl"/>
        <map:serialize type="fo2pdf"/>
      </map:match>
    </map:pipeline>

I know, this is my problem and not forrest's, but adding the fo2pdf serializer is something that people need to be aware of when upgrading from 0.6 to 0.7. But I guess http://forrest.apache.org/0.7/docs/faq.html#docbook a sufficient hint...

Ross Gardler made changes - 21/Oct/05 07:20 PM
Fix Version/s 0.8-dev [ 12310040 ]
Repository Revision Date User Message
ASF #481481 Sat Dec 02 01:24:20 UTC 2006 crossley Add a note that documentation has not been updated regarding sitemap examples and plugin sitemaps.
Issues FOR-546 and FOR-922
Files Changed
MODIFY /forrest/trunk/site-author/content/xdocs/docs_0_80/upgrading_08.xml

Repository Revision Date User Message
ASF #513074 Thu Mar 01 00:06:08 UTC 2007 thorsten FOR-546 - Sitemap reference doc should be updated to reflect plugin architecture
Files Changed
MODIFY /forrest/trunk/site-author/content/xdocs/docs_0_80/sitemap-ref.xml

Thorsten Scherler added a comment - 01/Mar/07 12:06 AM
Added note to look into the plugin.

Thorsten Scherler made changes - 01/Mar/07 12:06 AM
Status Open [ 1 ] Closed [ 6 ]
Resolution Fixed [ 1 ]
David Crossley added a comment - 01/Mar/07 02:53 AM
I think that you closed this issue too early Thorsten. Please see Ross' comments on 18/Jun/05 ... the whole document needs attention. I added a note to issue FOR-922 to cover it.