multiple page problem

Bug #890870 reported by Reto Knaak
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
chm2pdf
Unknown
Unknown
chm2pdf (Ubuntu)
Fix Released
Critical
Anders

Bug Description

As I am not familiar with linux, but needed to use chm2pdf soI installed Ubuntu 11.10 on my PC and I got the CHM2PDF 0.9.1.1ubuntu5 from the software center - this worked flawless.
Converting my chm file, some pages where there more than once in the output (one time for every link to that page ?)
I see in the script that something is done to Avoid duplicates in the list of image URLs (in class ImageCatcher), but nothing similar is done for class PageLister... I modified this in Pagelister class and Multiple pages seems now fixed:
                # Avoid duplicates in the list of URLs.
                if not self.pages.count('/'+value):
                    self.pages.append('/'+value)

Steve Stalcup (vorian)
Changed in chm2pdf (Ubuntu):
status: New → Confirmed
importance: Undecided → Critical
Revision history for this message
Reto Knaak (reto-knaak) wrote :

This is the first time I try to generate a patch... hope this is correct!

Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "Multiple page problem patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-sponsors please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Revision history for this message
Reto Knaak (reto-knaak) wrote :

There was a small error: forget to delete a line.
Now this patch works!

Revision history for this message
Matt Fischer (mfisch) wrote :

What's the full command line you used with that example help file?

Revision history for this message
Matt Fischer (mfisch) wrote :

nevermind, I found it!

Matt Fischer (mfisch)
tags: added: patch-forwarded-upstream
Anders (eddiedog988)
Changed in chm2pdf (Ubuntu):
assignee: nobody → Anders (eddiedog988)
madbiologist (me-again)
tags: added: oneiric
Yo (yleduc)
Changed in chm2pdf (Ubuntu):
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.