Closed Bug 812210 Opened 12 years ago Closed 11 years ago

m.doctissimo.fr uses functions in non-toplevel/global positions

Categories

(Tech Evangelism Graveyard :: French, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jdm, Unassigned)

References

()

Details

This code is present in http://m.doctissimo.fr/libs-59.js: >if(!window.dev&&!/.*(app\/debug\/).*/.test(window.location)&&!getCookie("MD_DEV_WEBAPP") { > if(!$.os.iphone&&!$.os.ipod&&!$.os.android){redir()} > if($.os.android) { > var reg=/^[^01]/; > if(!reg.test($.os.version)){redir()} > } > function redir(){...} > ... >} This is disallowed by the spec, but Webkit chooses to turn it into something that runs as a developer might expect. Gecko turns it into a ReferenceError (redir is not defined); there's bug 585536 open about doing something else with this construct, but no movement there yet. Therefore, it would be in Doctissimo's interest to correct this to something that is actually specified as working.
Did anybody contact them already? BTW, you can set Platform to All, this is not ARM-only.
I suspect nobody has done so. Feel free.
Hardware: ARM → All
If I'm correct, this issue can be solved by just rewriting the line with `function redir(){` to: ``` redir = function() { ``` (… and equally any similar occurences of this pattern.) I've contacted them with a rather generic (and English) email (sent to contact@d….fr as well as the Hex-email I found in the DNS Lookup). Let's see if it ever reaches a developer at Doctissimo …
This seems to be fixed.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in before you can comment on or make changes to this bug.