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)
Tech Evangelism Graveyard
French
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.
Reporter | ||
Updated•12 years ago
|
Comment 1•12 years ago
|
||
Did anybody contact them already?
BTW, you can set Platform to All, this is not ARM-only.
Comment 3•12 years ago
|
||
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 …
Comment 4•11 years ago
|
||
This seems to be fixed.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•