Roan Kattouw <> changed:

           What    |Removed                     |Added
             Status|REOPENED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #4 from Roan Kattouw <> 2010-11-05 20:36:38 
UTC ---
(In reply to comment #3)
> Are you sure this is enough? The behaviour seems to differ a bit as you also
> read "debug" state on line 2410 and 2579 and you don't read from the cookie
> there.
Good point, I missed those. Fixed in r76138.

 The cookie state is also disregarded in the __construct of
> ResourceLoaderContext. Maybe not a big difference now, but it seems to be a
> point where things can go bad in the future (where debug might mean something
> more). What I'm saying is debugging with a cookie and with a param is 
> different
> even now and might be worse in future. This doesn't feel like an expected or
> intended behaviour.
This is actually not a problem. All the debug cookie and the debug param when
applied to *index.php* do is make OutputPage.php generate links to load.php
with debug=true instead of debug=false. load.php itself never needs to read the
cookie for this reason (and if it did, that would violate cacheability).

Configure bugmail:
------- You are receiving this mail because: -------
You are on the CC list for the bug.

Wikibugs-l mailing list

Reply via email to