| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
weren't flipped if negative values were given. Trevor found this by running the original CSSJanus test suite against his NodeJS port; I should port the test suite to PHPUnit some time
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/110587
|
|
|
|
|
|
|
| |
Gilles van den Hoven
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/110557
|
|
|
|
|
|
|
|
| |
Incremented ResourceLoader::filterCacheVersion rather than decrementing to avoid potential confusion, especially since we already needed the incr.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/103988
|
|
|
|
|
|
|
| |
filter cache key, so broken entries stuck in cache will be reloaded.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/103979
|
|
|
|
|
|
|
| |
Reverts r103931 test change.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/103940
|
|
|
|
|
|
|
| |
Added some tests
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/103937
|
|
|
|
|
|
|
| |
Adjusting the tests, since it is now adding the newline after the number.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/103931
|
|
|
|
|
|
|
|
|
| |
It is apparently legal to have a decimal point without after it,
eg. var a = 5.
Not for hexadecimal numbers.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/103928
|
|
|
|
|
|
|
| |
Also detect as an error 1..0 or 1eeeeee5
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/103915
|
|
|
|
| |
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/103910
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
split over lines
This broke the OpenLayers support in the Maps extension, as used for example on TranslateWiki.net.
The original JavaScriptMinifier's tokenizer (r83885) explicitly didn't bother looking for the exponent part because it "didn't matter" to its internal state machine; however since r83891 added a max line length that definitely is not true.
I've split out handling of hex and decimal numerals, and let the decimal numeral handling check for exponents.
PHPUnit test cases were added in r103846.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/103865
|
|
|
|
| |
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/100783
|
|
|
|
|
|
|
|
|
|
|
| |
Documentation
Fix "/*" comments to "/**"
Flesh out some missing returns, change some return types
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/99808
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
valid Unicode letter characters
Followup r91591, r93020: patch to jsminplus to support Unicode chars and char escapes in identifiers
Fast-path check keeps runtime about the same on most scripts (eg jquery.js parsing was abround 4100ms both before and after on my test machine)
Slow-path code kicks in if plain ASCII word chars don't extend all the way to the next whitespace or punctuation char.
Using PCRE's Unicode properties magic to ensure that we're catching everything, following ECMA-262 edition 5.1 spec.
Note that identifiers using escapes don't get normalized to their UTF-8 form; this might be a nice thing to do as it saves a couple bytes, but currently there's no change made to output.
Added QUnit tests to verify that unicode letter & escapes work in identifiers in all supported browsers (ok back to IE 6, yay)
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/98281
|
|
|
|
|
|
|
| |
and r93820 caused them to get out of sync.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/94456
|
|
|
|
| |
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/93822
|
|
|
|
| |
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/93821
|
|
|
|
| |
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/93820
|
|
|
|
|
|
|
| |
contributions (i.e. changes other than very minor whitespace and Doxygen changes) by anyone other than me.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/93562
|
|
|
|
|
|
|
| |
This upstream release incorporates r92560 and r92563
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/93020
|
|
|
|
|
|
|
| |
could've theoretically resulted in very strange bugs) and only call wfExpandUrl() if available (the file is in includes/libs so it should work outside of MediaWiki)
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/92580
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
of the file.
This fixes the Notices that would be produced if several JSTokenizer were instantiated.
jsminplus was added in r91591.
Defines code was generated with:
$js = new JSTokenizer();
foreach ($js->opTypeNames as $operand => $name) echo "define('OP_$name', '$operand');\n";
foreach ($js->keywords as $keyword) echo "define('KEYWORD_" . strtoupper($keyword) . "', '$keyword');\n";
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/92563
|
|
|
|
|
|
|
| |
This should be uncontroversial.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/92560
|
|
|
|
|
|
|
|
|
|
|
| |
JavaScriptMinifierTest unit test cases. Had to change some of the test cases because they were not valid JavaScript programs -- one of the quoting tests was incorrectly quoted in the PHP sources, and several tests around return, continue, and break keywords failed due to not using them in the contexts that they require.
http://crisp.tweakblogs.net/blog/1856/jsmin+-version-13.html
JSMin+ under MPL 1.1/ GPL 2.0 / LGPL 2.1 license.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/91591
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
OutputPage::getStatusMessage() is a method to convert a numeric HTTP status
code to an english message. It does not really belong to the OutputPage were
it was for historical reason.
This patch move the basic function to a non MediaWiki dependant class in our
directory includes/libs. We could potentially enhances it, but I do not see
any use cases for us yet.
I have renamed the method to the shorter 'getMessage' since the word 'status'
is now in the class name.
Summary of changes:
* OutputPage::getStatusMessage becomes HttpStatus::getMessage
* Method moved to the new includes/libs/HttpStatus.php
* Autoloader updated
* History kept by using 'svn copy'
* No functional changes
* No input/output format changes
* Old occurences modified in phase3
* OutputPage::getStatusMessages() marked as deprecated
PHPUnit test suite is fine.
Tested manually using a redirection
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/91425
|
|
|
|
|
|
|
|
|
| |
Trimming trailing whitespace also
Doing in 3 commits, so hopefully reviewable in CR...
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/90280
|
|
|
|
|
|
|
|
|
| |
* Moved most of the bug 28235 code out to a separate library class, since I was running out of distinct function names.
* Merged the QUERY_STRING and PATH_INFO security checks, since they are dealing with the exact same problem. Removed WebRequest::isQueryStringBad().
* Deal with img_auth.php by having it specify what extension it expects to be streaming out. This extension can then be compared with the extension that IE might detect.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/89558
|
|
|
|
|
|
|
|
|
|
| |
CSSMin::minify()'s processing of url references for path adjustment or embedding had excluded explicit fully-qualified http: and https: URLs, but was damaging others such as data: URIs that were actually hardcoded into the original CSS.
This was affecting styles used on no.wikipedia.org which embedded a few icons directly into the style sheet.
Now checking for any URL scheme rather than hardcoding a check for http & https.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/87840
|
|
|
|
| |
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/86385
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* (bug 28591) Update/replace/supplement spyc (YAML parsing library)
* YAML API output is now 1.2 compliant, using JSON as the formatter
YAML 1.2 spec is a JSON subset - "The primary objective of this revision is to bring YAML into compliance with JSON as an official subset. YAML 1.2 is compatible with 1.1 for most practical applications - this is a minor revision." [1] Per discussion with Tim, switch YAML to use the JSON formatter
Was originally going to delete the ApiFormatYaml per Tim, but class needed to keep nicer (and apparent) output in API help page
Hence made subclass ApiFormatJson, minimal method overriding
spyc.php deleted from libs
[1] http://www.yaml.org/spec/1.2/spec.html#id2803629
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/86302
|
|
|
|
|
|
|
| |
available in production php5.2.4
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/83998
|
|
|
|
|
|
|
| |
which is what we run in production. Using equivalent hexadecimal
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/83997
|
|
|
|
| |
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/83989
|
|
|
|
| |
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/83988
|
|
|
|
|
|
|
| |
Copperman
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/83934
|
|
|
|
|
|
|
| |
apparently resolves to foo/bar rather than foo/../bar or bar . Fixed by eliminating duplicate slashes from the file URL when remapping
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/83902
|
|
|
|
|
|
|
| |
(each statement on its own line) in JavaScriptMinifier. Also add globals for these things and update minify.php for these new config vars.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/83891
|
|
|
|
| |
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/83885
|
|
|
|
| |
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/82457
|
|
|
|
|
|
|
| |
properly in next commit
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/82456
|
|
|
|
| |
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/82401
|
|
|
|
| |
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/82400
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Removed some repeated subexpressions, /(.)*/ etc. These cause the stack space to be rapidly exhausted, leading to a segfault for malicious input. I replaced the ([\r\n]|.) subexpressions with a dot with a /s modifier. Added a pcre.recursion_limit hack to take care of the rest.
* Supported assertions and non-capturing subpatterns in ParseMaster::add() by recognising that parentheses that aren't followed by "?" are capturing. Used another assertion to do this.
* Fixed a bug whereby if a single-line comment had a slash in it, it was recognised as a regex instead of a comment. This occurred because the leading whitespace caused the regex regex to match at an earlier string position than the comment regex, giving it undue precedence. This was the subject of several reports on IRC and the main reason for me starting work on JavaScriptDistiller. Used a lookbehind assertion.
* Give comments precedence over regexes and strings where there is ambiguity at the same string location. Not sure if this does anything, but it seemed like a good idea at the time.
* Removed unused variable ParseMaster::$TRIM.
* To test it, I ran the old and new jquery.js output through Google Closure Compiler. The result was the same, proving that there are no functional differences.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/82399
|
|
|
|
|
|
|
| |
the original if, it's redundant
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/82388
|
|
|
|
| |
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/82387
|
|
|
|
|
|
|
|
|
|
| |
the input to ParseMaster::add(), to avoid confusion over the lack of a /x modifier in that input.
* Broke up several regexes and added comments, so that humans might be able to understand them.
* Tested md5(minify(jquery.js)) as before. Checked speed, it's the same.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/82384
|
|
|
|
|
|
|
| |
possible. Tested by ensuring that the minifier output is the same before and after, with jquery.js as input.
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/82380
|
|
|
|
| |
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/82357
|
|
|
|
|
|
|
| |
it thought they were regexes. Make the regex regex a little bit stricter again, forbidding a regex from starting with * and other invalid things that were disallowed prior to r82344 (but still dropping / ) and forbidding multiline regexes
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/82352
|
|
|
|
|
|
|
| |
wrongly collapsed. Fixed by changing regex to be more flexible in protecting regexes; nothing wrong with being overinclusive in what we're protecting
Notes:
http://mediawiki.org/wiki/Special:Code/MediaWiki/82344
|