738344 - Pdf.js can't show "ä" and "ö" in Nordea bank account statements

Status: RESOLVED WORKSFORME

(Firefox :: PDF Viewer, defect)

Description

[JK]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:12.0) Gecko/20100101 Firefox/12.0
Build ID: 20120314195616

Steps to reproduce:

Pdf.js can't show "ä" and "ö" in Nordea bank account statements. It shows a space instead.

Comment 1

[JK]

Attachment: Screenshot: pdf.js

Comment 2

[JK]

Attachment: Screenshot: Adobe Reader (expected result)

Comment 3

[Jesper Hansen]

Report bugs to https://github.com/mozilla/pdf.js/issues as it is clearly stated on AMO

Comment 4

[JK]

(In reply to Jesper Hansen from comment #3)
> Report bugs to https://github.com/mozilla/pdf.js/issues as it is clearly
> stated on AMO

Why? See:

https://hg.mozilla.org/mozilla-central/rev/fa1c93e7f34f
https://hg.mozilla.org/mozilla-central/rev/4208fe045a06
https://hg.mozilla.org/mozilla-central/rev/0c784e5f33b9
https://hg.mozilla.org/mozilla-central/rev/e95e2d36dbe1

Invalid invalid status. It's a mozilla-central component.

Comment 5

[Josh Matthews [:jdm]]

mozilla-central just integrates all of the code at various times. All of the development work happens on github, and the developers don't look at bugzilla.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

(In reply to Josh Matthews [:jdm] from comment #5)
> mozilla-central just integrates all of the code at various times. All of the
> development work happens on github, and the developers don't look at
> bugzilla.

This is something that should probably be brought up in dev.planning. There isn't a ton of precedent for something like this. I'll note that I can think of no other code in Firefox where bug reports at BMO are closed as invalid, though.

Comment 7

[Magnus Melin [:mkmelin]]

Agreed that sounds strange for a shipped component.

Comment 8

[Dão Gottwald [:dao]]

(In reply to Ryan VanderMeulen from comment #6)
> I'll note that I can think
> of no other code in Firefox where bug reports at BMO are closed as invalid,
> though.

Yep, bugzilla does track upstream bugs. If nothing else, it's needed for product management and release drivers to track this feature, e.g. in order to disable it on release branches as long as it has major outstanding bugs.

The respective github issues should be linked to in the "See Also" field.

Comment 9

[Yury Delendik (:yury)]

It is a duplicate of https://github.com/mozilla/pdf.js/issues/1309, which is closed by https://github.com/mozilla/pdf.js/pull/1361

Comment 10

[Magnus Melin [:mkmelin]]

> The respective github issues should be linked to in the "See Also" field.

Bugzilla wont let us do that...

Comment 11

[Dão Gottwald [:dao]]

(In reply to Magnus Melin from comment #10)
> > The respective github issues should be linked to in the "See Also" field.
> 
> Bugzilla wont let us do that...

We should probably get that fixed.

Comment 12

[antistress]

Same problem with a french PDF and the "É" letter
see that document http://www.dicosmo.org/HoldUp/HoldUpPlanetaire.pdf : in the 1st page the title should be "LE HOLD-UP PLANÉTAIRE" but it shows "LE HOLD-UP PLAN TAIRE" instead

Comment 13

[Scoobidiver (away)]

It works for me with http://www.lufthansa.com/mediapool/pdf/31/media_907231.pdf in PDF Viewer 0.2.432.
JK, is your issue fixed in that version?

Comment 14

[antistress]

isn't Bug 738918 a dup ?

Comment 15

[Scoobidiver (away)]

(In reply to antistress from comment #14)
> isn't Bug 738918 a dup ?
This bug is fixed in PDF Viewer 0.2.432 based on the testcase in https://github.com/mozilla/pdf.js/issues/1309, not bug 738918.

Comment 16

[antistress]

Sorry i'm not sure to understand. None of them are marked as "fixed", and i don't know if i must filled a 3red bug for 738344#c12 ?
Also, both should be marked as Blocking Bug 714712 i think ?

Comment 17

[Yury Delendik (:yury)]

Not sure if they have to block 714712. PDF Viewer is already part of the mozilla-central. Will just using PDF Viewer compartment be enough?

Also, comment 12 is bug 738918

Comment 18

[:Gavin Sharp [email: [e-mail]]]

(In reply to Dão Gottwald [:dao] from comment #11)
> > Bugzilla wont let us do that...
> 
> We should probably get that fixed.

Bug 739091.

Comment 19

[JK]

(In reply to Scoobidiver from comment #13)
> It works for me with
> http://www.lufthansa.com/mediapool/pdf/31/media_907231.pdf in PDF Viewer
> 0.2.432.
> JK, is your issue fixed in that version?

That file works, but the Nordea bank statements don't.

Comment 20

[Scoobidiver (away)]

(In reply to JK from comment #19)
> That file works, but the Nordea bank statements don't.
So it's not issue 1309.

Comment 21

[Scoobidiver (away)]

JK, can you provide a link to an affected PDF document where it's not required to be logged in?

Comment 22

[Thomas Ahlblom]

JK, any update on comment 21?

And does the problem still exist in 0.2.536 or later?

Comment 23

[JK]

(In reply to Scoobidiver from comment #21)
> JK, can you provide a link to an affected PDF document where it's not
> required to be logged in?

I suppose you could ask the Nordea administrators for a sample statement that's nobody's personal account statement.