Tim Starling [Thu, 17 Oct 2013 00:03:01 +0000 (00:03 +0000)]
Resubmit "Add support for mysqli extension"
This reverts commit
1d7e12179df0db332f9ac139e8b7d831e04a6a10.
Change-Id: Id1a3fc34e94006b880f58de83e5e7c70b21d26ed
jenkins-bot [Thu, 17 Oct 2013 00:37:44 +0000 (00:37 +0000)]
Merge "Revert "Add new recentchanges field rc_source to replace rc_type""
Reedy [Thu, 17 Oct 2013 00:32:40 +0000 (00:32 +0000)]
Revert "Add new recentchanges field rc_source to replace rc_type"
No feature flag, and also not deployed on WMF wikis. Asking for schema changes by the time we branch tomorrow is just unfair
This reverts commit
e478c230f6d3f1b7638d4443682eff0df7650def.
Change-Id: I125c30e76fa04679cc04ae7b8728ae3a61f35194
jrobson [Wed, 16 Oct 2013 15:19:34 +0000 (08:19 -0700)]
Vector: Fix font size regression
As Krinkle noted the font size of the body content ballooned with
Ica0b69ad95051e15cd8ed9888927e034c13fbf92 - this rectifies it.
Change-Id: I3a5a159cbb04a1a24277596ca304805a84d3b77f
jenkins-bot [Wed, 16 Oct 2013 23:54:05 +0000 (23:54 +0000)]
Merge "Revert "Add support for mysqli extension""
Tim Starling [Wed, 16 Oct 2013 23:49:24 +0000 (23:49 +0000)]
Revert "Add support for mysqli extension"
Throws exception unconditionally, installer support is totally broken.
This reverts commit
b2d64432ee02661ea29cb727686e10e7a4980dfb.
Change-Id: I2edb78440806754c42d48c5da2ff74b4cb40fa2d
jenkins-bot [Wed, 16 Oct 2013 23:35:20 +0000 (23:35 +0000)]
Merge "Added a "syncviadelete" param to copyFileBackend script"
jenkins-bot [Wed, 16 Oct 2013 23:33:37 +0000 (23:33 +0000)]
Merge "Add new recentchanges field rc_source to replace rc_type"
Aaron Schulz [Tue, 13 Aug 2013 17:21:50 +0000 (10:21 -0700)]
Added a "syncviadelete" param to copyFileBackend script
* Added a bit of tabbing to clarify the output
Change-Id: I4b0095595d742452b151e6ed6765a9dc805a55ae
Jakub Vrana [Fri, 3 May 2013 06:01:32 +0000 (23:01 -0700)]
Add support for mysqli extension
DatabaseBase::factory()'s parameter array now takes an optional
'driver' variable to allow specifying a specific driver for the
database class if we support more than one.
This is so we can gracefully support mysql/mysqli as the former is
deprecated, but the concept could be extended to other databases
as well.
Bug: 45288
Co-Author: Chad Horohoe <chadh@wikimedia.org>
Change-Id: I6733fe21c4aa7443e409c5dfa7c789552b2b62b7
Translation updater bot [Wed, 16 Oct 2013 21:32:50 +0000 (21:32 +0000)]
Localisation updates from translatewiki.net.
Change-Id: Id8e5920d1a5f58fc904a812369df35fe2f8f448e
jenkins-bot [Wed, 16 Oct 2013 18:40:10 +0000 (18:40 +0000)]
Merge "Keep returnto, returntoquery on Special:CreateAccount redirect"
Erik Bernhardson [Mon, 23 Sep 2013 22:38:01 +0000 (15:38 -0700)]
Add new recentchanges field rc_source to replace rc_type
The existing field to differentiate between kinds of recentchanges rows is
the rc_type field. We want to allow extensions to insert their own custom
data into recentchanges, but we have learned via the NS_* series of constants
that requiring extensions to "register" a specific number is very error prone.
The solution, which this commit implements the first phase of, is to utilize
a new 16 byte string field rc_source. Within that field change types will be
prefixed strings such as 'mw.edit' and 'mw.new'.
This commit adds the new field and begins populating it with data. At some
point in the future the rc_type field will be dropped. While WMF wiki's will
simply wait out the 30 day recentchanges history, other wiki's have the option
of letting update.php populate rc_source, or manually applying the db change and
utilizing the PopulateRecentChangeSource maintenance script.
Change-Id: Iaddd6c446373a68d31586ed54346db7d04e13b2c
jenkins-bot [Wed, 16 Oct 2013 18:12:33 +0000 (18:12 +0000)]
Merge "Use max() for job count for sanity"
jenkins-bot [Wed, 16 Oct 2013 17:31:50 +0000 (17:31 +0000)]
Merge "Support rate-limiting thumbnail generation"
jenkins-bot [Wed, 16 Oct 2013 17:29:00 +0000 (17:29 +0000)]
Merge "Move noflip annotations into rules themselves"
jrobson [Wed, 16 Oct 2013 15:12:34 +0000 (08:12 -0700)]
Move noflip annotations into rules themselves
In LESS these seem to be honoured better inside the rule.
Bug: 55779
Change-Id: I412575f37dd647d018d46b96f5642f5d68ebeb74
jenkins-bot [Wed, 16 Oct 2013 17:08:13 +0000 (17:08 +0000)]
Merge "Include miscellaneous query parameters in $wgSecureLogin redirect"
jenkins-bot [Wed, 16 Oct 2013 16:51:12 +0000 (16:51 +0000)]
Merge "Remove unnecessary defined( 'MEDIAWIKI' ) check"
jenkins-bot [Wed, 16 Oct 2013 16:47:54 +0000 (16:47 +0000)]
Merge "Reindent UsercreateTemplate and UserloginTemplate"
jenkins-bot [Wed, 16 Oct 2013 16:45:48 +0000 (16:45 +0000)]
Merge "Add a line of explanation to Html::expandAttributes"
Bartosz Dziewoński [Wed, 16 Oct 2013 16:40:40 +0000 (18:40 +0200)]
Remove unnecessary defined( 'MEDIAWIKI' ) check
This file doesn't run any code, it just defines a class.
Change-Id: I92664e2c1a5617a28c0fb22c1d1fd99a46224a04
Bartosz Dziewoński [Wed, 16 Oct 2013 16:24:12 +0000 (18:24 +0200)]
Reindent UsercreateTemplate and UserloginTemplate
A perceptive reviewer will note several issues with this code,
including at least one outright bug. I intend to fix them in
subsequent commit; I tried to avoid any non-whitespace changes
here to make it at all possible to review this.
This file has several lines longer than the phpcs limit of
100 characters, and several cases of the <?php } ?> pattern
phpcs complains about. I don't think it makes sense to enforce
these rules here; the code would IMO become less readable.
Change-Id: I95ab0f644fd5963878018f483f22e650867b5135
Bartosz Dziewoński [Wed, 16 Oct 2013 15:42:07 +0000 (17:42 +0200)]
Add a line of explanation to Html::expandAttributes
Change-Id: I7ad453638075a5e875c6c97df10690747c2488f0
Brad Jorsch [Wed, 16 Oct 2013 14:29:39 +0000 (10:29 -0400)]
Include miscellaneous query parameters in $wgSecureLogin redirect
The redirect from http to https should include any extra query
parameters, such as the ones for "campaigns".
But we still want to override returnto and returntoquery (in case they
were changed in ::load(), e.g. lines 139–140), and we omit any 'title'
parameter that may have been passed.
Bug: 55761
Change-Id: Icc8b4132e8682d782b0c049631328b94745eb355
shirayuki [Wed, 16 Oct 2013 13:10:02 +0000 (22:10 +0900)]
Fix double-space in messages
Change-Id: I5dbc0512b6447dfeb7c287e212245e86d105928b
Matthew Flaschen [Wed, 16 Oct 2013 07:32:21 +0000 (03:32 -0400)]
Keep returnto, returntoquery on Special:CreateAccount redirect
Bug: 55765
Change-Id: I1d05a1b91b2ac4097e09d11510d53fdfce1e0565
Erich Lerch [Tue, 15 Oct 2013 23:09:27 +0000 (01:09 +0200)]
Escape backslashes in image metadata
Uploading a file with backslashes in the metadata
causes database exception on PostgreSQL, because
backslashes are sent to the 'bytea' datatype unencoded.
Bug: 52017
Change-Id: Ife56cffe3df83ec1a7bd39cdc4a0489f23354494
jenkins-bot [Tue, 15 Oct 2013 22:16:48 +0000 (22:16 +0000)]
Merge "In maintenance/postgres/tables.sql, the profiling table should be defined like this:"
Jeff Janes [Tue, 15 Oct 2013 20:19:10 +0000 (20:19 +0000)]
In maintenance/postgres/tables.sql, the profiling table should be defined like
this:
CREATE TABLE profiling (
pf_count INTEGER NOT NULL DEFAULT 0,
pf_time FLOAT NOT NULL DEFAULT 0,
pf_memory FLOAT NOT NULL DEFAULT 0,
pf_name TEXT NOT NULL,
pf_server TEXT NULL
);
The current use of NUMERIC(18,10) very rapidly overflows the pf_memory column,
generating errors. Also, the NUMERIC is very much slower than float, and in
this case it has no advantages.
Bug: 55722
Change-Id: I48b00d55aaed821a4ceb9365033817a3b477d71a
Timo Tijhof [Tue, 15 Oct 2013 20:09:54 +0000 (22:09 +0200)]
mediawiki.Title: Fix doc reference error
Follows-up
165373d which introduced a reference to a browser
global not in our index (causing a jsduck error).
Unless we're using an element that has special DOM methods I
don't think it's worth adding element subclasses to the doc index.
We generally just use HTMLElement so changed it to that.
Also made a minor optimisation in merging the img/src assignments
by using .prop( 'src' ) and .src directly. And while at it also
reduced it to direct access with [0] instead of .get().
Change-Id: I3a521eda0aeeb7847a96d474316d1d5e6e8917d1
Translation updater bot [Tue, 15 Oct 2013 19:19:21 +0000 (19:19 +0000)]
Localisation updates from translatewiki.net.
Change-Id: Ie45e695bf4dc5fc8177ed33e400f7ca2000519a0
jenkins-bot [Tue, 15 Oct 2013 18:39:08 +0000 (18:39 +0000)]
Merge "Remove comment which is plainly wrong"
jeroendedauw [Tue, 15 Oct 2013 18:31:34 +0000 (20:31 +0200)]
Remove comment which is plainly wrong
Change-Id: I89bb9243a19e0355260e4e1c217e3868014c4886
jenkins-bot [Tue, 15 Oct 2013 17:53:12 +0000 (17:53 +0000)]
Merge "Expose changed links in LinksUpdate"
Nik Everett [Wed, 9 Oct 2013 18:48:37 +0000 (18:48 +0000)]
Expose changed links in LinksUpdate
These links are only really available during the LinksUpdateComplete
hook.
To make sure they are always available this also removes dump links
updates which would prevent this data from being built.
Change-Id: I115f8cf9d3b0a1dba979ceb58b5f14dd0b76ec49
S Page [Wed, 26 Jun 2013 06:03:17 +0000 (23:03 -0700)]
Add vform format to HTMLForm, use in PasswordReset
This adds a new 'vform' display format to HTMLForm which styles forms
using it in the new compact stacked vertical format:
* Applies class mw-ui-vform to the form.
* Gives Submit button 'mw-ui-{button,big,block,primary}' classes and
spaced block styling.
* Styles the error class.
* HTMLForm divs are too nested to get styling "for free", so add
.mw-ui-vform-div selector to Agora CSS and apply this class to form
divs (maybe not ideal fix).
* HTMLCheckField nests the checkbox inside label.
* Add method to setShowEmptyLabel(); vform format sets this false as it
doesn't want generated HTML for empty labels adding vertical space.
In FormSpecialPage, don't set the wrapper legend with its old-school
line around the form if in vform style.
Build new mediawiki.ui CSS with the .mw-ui-vform-div styling
Special:PasswordReset requests the new styling with setDisplayFormat( 'vform' )
and turns off the wrapper legend.
(This version of the patch doesn't switch the default format of HTMLForm,
individual forms have to choose it.)
Notes are at
https://www.mediawiki.org/wiki/Agora/Engineering#HTMLForm_Issues
Change-Id: Id03d185bbee990595bfc469a61163cc598fc3441
jenkins-bot [Tue, 15 Oct 2013 16:34:44 +0000 (16:34 +0000)]
Merge "Vector: New beta module with new typography styles"
Bryan Davis [Fri, 11 Oct 2013 19:42:10 +0000 (13:42 -0600)]
Add configurable delay between purgeChangedPages batches
UDP flooding is possible when purging large numbers of files. This
change adds a command line switch to the purgeChangedPages.php
maintenance script to insert an artificial delay between purge batches.
The intent is to allow the network time to process the batch that was
just sent before flooding with a second batch.
`--sleep-per-batch` can be used in combination with `--batch-size` to
manage network and to some extent query load. A suggested starting point
would be to use `--sleep-per-batch=500` in concert with the default
batch size of 100 to target a maximum rate of 200 packets/second.
Bug: 55632
Change-Id: Ibfc54b1767f145098465404a2b23cd92852e41fd
Aaron Schulz [Fri, 31 May 2013 23:12:18 +0000 (16:12 -0700)]
Simplified IPv6 regex per comment
* We already require PHP 5.3.2, which includes PCRE 7.2+.
PHP 5.2.4 includes PCRE 7.2 per http://php.net/ChangeLog-5.php.
Change-Id: Ie9c1a0a93bef30a3a67e0140e621814351f8cea1
Timo Tijhof [Tue, 8 Oct 2013 21:47:42 +0000 (23:47 +0200)]
resourceloader: Don't catch LESS error in ResourceLoaderFileModule
Instead let it throw up to ResourceLoader::makeModuleResponse.. By using
the central logic for exception handling the parse error becomes more
discoverable and obvious.
* The module is marked as in "error" state (currently it would still serve
the result of file A and C, even if file B has a parse error).
This matches the behaviour of the javascript checker we have.
* The exception comment is produced on top of the load.php response
(where users expect it) instead of buried somewhere between the
different modules and the different files within the modules.
* The exception comment will not be stripped out by CSSMin.
* DRY :)
Without this, debugging less parse errors is harder than it should be.
Bug: 55442
Change-Id: I6510b43f02cf82568b7fd2421209a8e636d5e115
jenkins-bot [Mon, 14 Oct 2013 13:16:33 +0000 (13:16 +0000)]
Merge "Refresh Deletereason-dropdown"
jenkins-bot [Mon, 14 Oct 2013 13:14:35 +0000 (13:14 +0000)]
Merge "Add APIQuerySiteInfoStatisticsInfo hook"
Siebrand Mazeland [Mon, 7 Oct 2013 09:32:05 +0000 (11:32 +0200)]
Document class property and break long lines
Change-Id: Id391dc8911de8b6df0fb95cba58cffee972ca507
This, that and the other [Mon, 14 Oct 2013 10:54:12 +0000 (21:54 +1100)]
Refresh Deletereason-dropdown
I went around to a few MW installations on the web, and all that had
modified this message added "Spam". (Also per the bug.) And "Broken
redirect" seems a sensible, universal addition as well.
Also putting in order of "commonness" and fixing the inconsistent spacing
of the bulleted list - it annoys me every time.
Bug: 47573
Change-Id: If50f30df95f969f0b56a9058fc57e7c542b7215a
Translation updater bot [Mon, 14 Oct 2013 09:28:56 +0000 (09:28 +0000)]
Localisation updates from translatewiki.net.
Change-Id: I2e39d39032e0a7fb32b9f5385f95f3047bbdb264
Timo Tijhof [Wed, 9 Oct 2013 00:20:41 +0000 (02:20 +0200)]
exception: Move logging logic to static method of MWExceptionHandler
Follows-up
c28251a9fd, which unconditionally called logException
on objects that aren't always instances of MWException.
This makes it possible to, for example, log PHP errors or errors
from Less in ResourceLoader to be logged as well (which naturally
don't have a logException method).
As a result of the handling now being generic, non-MWException
errors will now be loggable, and also have their ID fixed between
HTML output and the debug log entry.
* Not yet removing MWException::getLogMessage (introduced in
bcb9f9e1c)
as it has been around since 2006 (released in 1.8.0).
* Not yet removing MWException::getLogId (introduced in
70841c5867)
as it was part of the 1.20.0 release.
* Removed MWException::logException (introduced in
c28251a9fd)
as it was only added a few weeks ago within this release cycle.
Change-Id: Iab98e3a7a9b78d8602e69e0571b35cf107a96b72
Timo Tijhof [Tue, 8 Oct 2013 00:40:24 +0000 (02:40 +0200)]
vector: Apply coding conventions to screen.less and minor cleanup
Follows-up
08cf0b1,
7e953bbe.
* Removed obsolete comment about @embed, since this file no longer
uses that.
* Removed ".less" per LESS convention.
* Made whitespace consistent with coding conventions.
No spaces inside .transition(), just like there are no spaces
in .background-image() or url().
* Line breaks between different rules.
Change-Id: I5956d179eb46f6ee4e0c313eafe008c434e61e31
Bartosz Dziewoński [Fri, 27 Sep 2013 20:52:44 +0000 (22:52 +0200)]
Remove calls to deprecated HTMLForm#setTitle
Use DerivativeContext with modified title instead.
Most of those should be using FormSpecialPage…
Change-Id: Ia66f3b4b22866cc4ac4ac7b1f2fc2a4b1a9e73f6
Bartosz Dziewoński [Sat, 12 Oct 2013 18:45:07 +0000 (20:45 +0200)]
Rename SpecialRecentChanges::addRecentChangesJS() to addModules()
…and make it protected. It makes no sense to include class name in the
function name, especially since subclasses might want to customize it.
Change-Id: Ic0d23f4400b88596fdb11f6c0d8aefc9aea589ba
Bartosz Dziewoński [Sat, 12 Oct 2013 00:41:47 +0000 (02:41 +0200)]
SpecialRecentChanges: Don't call getUser() repeatedly in getDefaultOptions()
Change-Id: If0ec42e89887c98968cd7878e45b1944e9fdf7ac
Bartosz Dziewoński [Fri, 11 Oct 2013 21:01:57 +0000 (23:01 +0200)]
SpecialWatchlist: Remove old irrelevant comment
The code it was commenting has been removed in r10652 in 2005…
Change-Id: I33846196c1f62800db2a37b6b03bdb47280af7d4
Bartosz Dziewoński [Fri, 11 Oct 2013 19:42:23 +0000 (21:42 +0200)]
SpecialWatchlist: Remove dead cutoff code
Added back in r1532, it got broken in one of the numerous refactors
since then and now only triggers when 'days' request parameter is
passed explicitly and is malformed.
Just use floatval() instead.
Change-Id: I20c1e816a11fca5037af93bf81b46839f2a5efb0
Translation updater bot [Sun, 13 Oct 2013 19:20:40 +0000 (19:20 +0000)]
Localisation updates from translatewiki.net.
Change-Id: I3b0c0763a0f2a7b40c4b25821cffac6b136ea6af
Kevin Israel [Sun, 13 Oct 2013 13:49:03 +0000 (09:49 -0400)]
Ensure that $call['args'] is set before using it
Fixes PHP Notice: Undefined index: args in /home/ki/Projects/mediawiki/
core/includes/Exception.php on line 755 when running code from
maintenance/eval.php .
Change-Id: Ic4a806232aadae0a60f5bc06de1604d9c5996e6f
umherirrender [Sat, 12 Oct 2013 20:48:26 +0000 (22:48 +0200)]
Pass Context to User::resetOptions
Change-Id: I0444777697ed9a8b8afdd0e5a96841848b162a7a
jenkins-bot [Sat, 12 Oct 2013 19:26:46 +0000 (19:26 +0000)]
Merge "Add bot to createAndPromote.php groups"
Translation updater bot [Sat, 12 Oct 2013 18:42:51 +0000 (18:42 +0000)]
Localisation updates from translatewiki.net.
Change-Id: I018c32bec9b2b04590657afd96606ed473cdfed6
jenkins-bot [Sat, 12 Oct 2013 17:20:38 +0000 (17:20 +0000)]
Merge "Revert visibility change to HTMLForm $typeMappings"
Siebrand Mazeland [Sat, 12 Oct 2013 15:35:26 +0000 (17:35 +0200)]
Update formatting
Change-Id: If3dd6a1ac4b16479244a847420f0488ba8c51e2b
Bartosz Dziewoński [Sat, 12 Oct 2013 14:56:41 +0000 (16:56 +0200)]
FormOptions: More documentation!
Change-Id: I088b07c1672426c45d995fdf96ac7fc7bda21307
S Page [Sat, 12 Oct 2013 08:10:24 +0000 (01:10 -0700)]
Revert visibility change to HTMLForm $typeMappings
Change Id5668141835e22e3e1b043109e50d73ccd6b1229 changed the visibility
of static $typeMappings to private. It turns out this breaks
http://www.blue-spice.org/ who want to add their own HTMLForm types and
have no way to extend or override this except by changing it. So set it
to public static.
As https://www.mediawiki.org/wiki/Manual:Coding_conventions/PHP says,
"It's generally a good idea to avoid visibility changes unless you're
making changes to the function which would break old uses of it anyway."
That trumps PHP CodeSniffer warnings.
Change-Id: Ife82596e96a4650b977422aae6b32f24a392d5fb
Kunal Mehta [Sat, 5 Oct 2013 21:32:08 +0000 (14:32 -0700)]
Add APIQuerySiteInfoStatisticsInfo hook
Allows extensions to modify the output of
action=query&meta=siteinfo&siprop=statstics
Accompanies the SpecialStatsAddExtra hook for Special:Statistics
Change-Id: I712748077a0ac2fb729bfe3b6616ac4981cb754e
Bryan Davis [Fri, 11 Oct 2013 22:41:59 +0000 (16:41 -0600)]
Modernize SquidUpdate php code
- Update to PHP5 object coding style
- Fix @param comments using wrong argument order
- Add some phpdoc comments
Change-Id: Ia3190a6276e807ebef4f4aaeedc0e39819dc992a
jenkins-bot [Fri, 11 Oct 2013 20:36:06 +0000 (20:36 +0000)]
Merge "Blacklist Nokia's Ovi Browser from ResourceLoader"
Max Semenik [Fri, 11 Oct 2013 15:49:04 +0000 (19:49 +0400)]
Blacklist Nokia's Ovi Browser from ResourceLoader
It runs scripts proxy-side much like Opera Mini does, resulting in nothing
resembling a real JS engine. This causes problems and the browser is popular
enough to warrant a block.
Bug: 55600
Change-Id: I2c1ee78828c2b9cead4f82ef795972a4c28140b4
Translation updater bot [Fri, 11 Oct 2013 19:10:32 +0000 (19:10 +0000)]
Localisation updates from translatewiki.net.
Change-Id: I8f4d44a3917bc7264d7dacc4b1e75641dab845c0
jenkins-bot [Fri, 11 Oct 2013 18:50:51 +0000 (18:50 +0000)]
Merge "exception: Account for $call['file'] and $call['line'] being unset"
Timo Tijhof [Fri, 11 Oct 2013 18:18:07 +0000 (20:18 +0200)]
exception: Account for $call['file'] and $call['line'] being unset
Bug: 55634
Change-Id: I1173216cade73216848816f6bb51e54096abdfde
jenkins-bot [Fri, 11 Oct 2013 18:28:29 +0000 (18:28 +0000)]
Merge "Add script to purge changed files from cache"
jenkins-bot [Fri, 11 Oct 2013 17:27:45 +0000 (17:27 +0000)]
Merge "mw.inspect: decline to report module sizes when in debug mode"
jenkins-bot [Fri, 11 Oct 2013 17:26:03 +0000 (17:26 +0000)]
Merge "FormatJson: minor cleanup"
jenkins-bot [Fri, 11 Oct 2013 16:31:10 +0000 (16:31 +0000)]
Merge "Add an explicit targets declaration for mediawiki.inspect & $.byteLength"
Ori Livneh [Fri, 11 Oct 2013 08:31:00 +0000 (01:31 -0700)]
Add an explicit targets declaration for mediawiki.inspect & $.byteLength
Jon Robson said[1] he'd be interested in making mediawiki.inspect usable on
the mobile site by specifying both mobile & desktop targets explicitly in the
resource declaration. That's what this patch does. The cost to this is very
minimal because the module is never loaded by default. mw.loader.inspect()
wraps the call to the inspector with an mw.loader.using() call that causes the
module to be retrieved. It is not retrieved otherwise.
[1]: http://lists.wikimedia.org/pipermail/wikitech-l/2013-October/072381.html
Change-Id: I7cf292d3b8fb554bd1fc2143ca45399efccb70aa
Ori Livneh [Fri, 11 Oct 2013 10:44:33 +0000 (03:44 -0700)]
mw.inspect: decline to report module sizes when in debug mode
- In debug mode, 1 resource = 1 request, so module size is easier to determine
via the network view of the developer tools in Firefox & WebKit browsers.
- Determining the module size in debug mode in JavaScript would require
modifying core ResourceLoader code or re-fetching each resource with a
separate XHR to measure its size.
- It's not worth it.
Bug: 55553
Change-Id: I73d1bb4141c1b6ed64a72fdc416048937a2e1146
Timo Tijhof [Sun, 6 Oct 2013 22:15:36 +0000 (00:15 +0200)]
mediawiki.page.gallery: Clean up documentation for hook
Follows-up
14b3f87.
Change-Id: Ia917516d985a6e6f6e4bbca9ea8b5437175a6aa8
jrobson [Thu, 3 Oct 2013 20:22:14 +0000 (13:22 -0700)]
Vector: New beta module with new typography styles
To enable install Extension:BetaFeatures and Extension:VectorBeta
in LocalSettings.php
Changes to core:
* Refactor screen.less to make use of variables
Introduce dormant skins.vector.beta module
* Activated by https://gerrit.wikimedia.org/r/#/admin/projects/mediawiki/extensions/VectorBeta
* Load different environment variables depending on whether in a beta
experiment or not
* Switch global fonts for text from browser default sans-serif to a
specific sans-serif stack.
* Switch global fonts for headings to Georgia/serif
* Cleanup over-specific specifiers in process
* Adjust styling of navigation menus to focus on content
* Adjust styling of collapsible navigation menu to be less prominent
See https://www.mediawiki.org/wiki/Wikimedia_Foundation_Design/Typography
Change-Id: Ica0b69ad95051e15cd8ed9888927e034c13fbf92
jenkins-bot [Thu, 10 Oct 2013 22:12:55 +0000 (22:12 +0000)]
Merge "Do not set wgServer in cli install unless explicitly passed"
Aaron Schulz [Thu, 10 Oct 2013 21:56:11 +0000 (14:56 -0700)]
Use max() for job count for sanity
Change-Id: If5979ccbede2dd70fcd26b7caae82419ddffe542
jenkins-bot [Thu, 10 Oct 2013 20:54:37 +0000 (20:54 +0000)]
Merge "Made JobQueueDB use getConnectionRef()"
Translation updater bot [Thu, 10 Oct 2013 19:03:14 +0000 (19:03 +0000)]
Localisation updates from translatewiki.net.
Change-Id: I6ed3b4bc205bb8b6a09aea841a73aa0d8e317554
jenkins-bot [Thu, 10 Oct 2013 18:08:25 +0000 (18:08 +0000)]
Merge "Handle raw <h#> when calculating $rawtoc"
YuviPanda [Sun, 6 Oct 2013 19:41:18 +0000 (01:11 +0530)]
Do not set wgServer in cli install unless explicitly passed
Currently, if we do not pass a wgServer value (via --server),
the CLI installer sets the value to the default value
specified in DefaultSettings.php - which is to 'guess' by
calling into WebRequest::detectServer. This yields terrible
results, since WebRequest::detectServer expects to be working
in the context of a Web Request - and hence with HTTP Host
header information. Since calling from the CLI does not give
it host header information, it falls back to 'localhost',
which is not the value you usually want.
If we just do not set wgServer when it is not specified,
it is automatically calculated on every request by
WebRequest::detectServer, which does a splendid job.
Bug: 55376
Change-Id: I5436dd8c340604cbb59406a507188e11c8f86e86
Chad Horohoe [Thu, 10 Oct 2013 01:14:42 +0000 (18:14 -0700)]
Remove silly test cases that are likely to fail
Nobody should be giving objects or arrays to addIdentifierQuotes() anyway
Change-Id: Ibebf5d32196ac5b80cd477aedfef982fd2c9a0e6
jenkins-bot [Wed, 9 Oct 2013 23:48:05 +0000 (23:48 +0000)]
Merge "Fixed trx isolation bug that could cause text not to be found"
Aaron Schulz [Mon, 7 Oct 2013 16:18:16 +0000 (09:18 -0700)]
Made JobQueueDB use getConnectionRef()
* Also moved insertFields() function up a bit about DB connection methods
Change-Id: I7a9d9c9bc71038fe280542b13e0c0e94bfc4de1e
jenkins-bot [Wed, 9 Oct 2013 22:21:20 +0000 (22:21 +0000)]
Merge "Made root job de-duplication work without cache setup"
jenkins-bot [Wed, 9 Oct 2013 22:12:26 +0000 (22:12 +0000)]
Merge "Escape backticks when quoting MySQL identifiers"
jenkins-bot [Wed, 9 Oct 2013 22:01:19 +0000 (22:01 +0000)]
Merge "Add mediawiki.inspect module"
Ori Livneh [Tue, 8 Oct 2013 06:08:54 +0000 (23:08 -0700)]
Add mediawiki.inspect module
This patch adds a mediawiki.inspect ResourceLoader module. The module contains
tools for inspecting page composition and performance. It is currently
restricted to a set of static methods for querying the ResourceLoader module
registry, the most useful of which is #inspectLoadedModules, which prints a
list of loaded ResourceLoader modules and their size in kilobytes, ordered from
largest to smallest. This method has a lazy-loading alias: mw.loader.inspect.
Change-Id: I5a007fdf11196664f425cd7800eec7fc3fdb629f
Bryan Davis [Tue, 8 Oct 2013 18:14:52 +0000 (12:14 -0600)]
Escape backticks when quoting MySQL identifiers
Previously mysql_real_escape_string() was improperly used which escapes
for inclusion in a plain string rather than an identifier.
Also adds basic test support for the DatabaseMysqlBase class.
Co-Authored by: Antoine Musso <hashar@free.fr>
Bug: 55427
Change-Id: Ic045e195c89d8d5d0f0edbda0cd1df781de7025c
Translation updater bot [Wed, 9 Oct 2013 19:36:07 +0000 (19:36 +0000)]
Localisation updates from translatewiki.net.
Change-Id: I91e6b1a32aaca18175c97130685a26e9a5704f8d
jenkins-bot [Wed, 9 Oct 2013 19:11:44 +0000 (19:11 +0000)]
Merge "Fix ForeignAPIRepo::fileExistsBatch()"
jenkins-bot [Wed, 9 Oct 2013 18:14:09 +0000 (18:14 +0000)]
Merge "Tests for API section editing"
Kunal Mehta [Sat, 7 Sep 2013 05:56:37 +0000 (22:56 -0700)]
Tests for API section editing
Change-Id: Ibc05c54cd2c9e1775f72dd969e7677b5238e12ad
jenkins-bot [Wed, 9 Oct 2013 18:00:43 +0000 (18:00 +0000)]
Merge "Remove wfDl() and cleanup everything it touched"
jenkins-bot [Wed, 9 Oct 2013 17:58:50 +0000 (17:58 +0000)]
Merge "Fix PHP CodeSniffer warnings and errors"
Chad Horohoe [Wed, 9 Oct 2013 16:46:57 +0000 (09:46 -0700)]
Remove wfDl() and cleanup everything it touched
wfDl() is a wrapper around dl(), which is an evil function and
basically only works from the command line of Zend. Luckily
no extension has ever used this thing, so let's just remove it
outright.
For comparison, here's a list of places it does not work:
- hhvm
- php as apache module
- php compiled with zts support
- safe_mode
- Basically any shared host that cares about security
Most callers are using it to check for extension support and are
actually failing gracefully when wfDl() returns false. In these
places we're just going to use extension_loaded().
While we're at it, clean up some of the test skip logic in the
media tests so we can bail as early as possible if we know we
can't complete the test.
This also immediately removes $wgLoadFileinfoExtension. It's been
enabled by default since 5.3 and falls back gracefully when the
support isn't available.
Change-Id: Ieb430dfc74483731dde51d6e20fa700d641ba1f4
Brad Jorsch [Wed, 9 Oct 2013 15:03:40 +0000 (11:03 -0400)]
Handle raw <h#> when calculating $rawtoc
When the parser is constructing $rawtoc, it needs the sectionIndex
number to be able to calculate the byteoffset. This number is only
available for wikitext headings ("== foo =="), HTML headings
("<h2>foo</h2>") do not have it and the lack makes byteoffset be wrong
for all subsequent headings in the page.
To fix this, we just omit output of byteoffset in this situation.
Bug: 25203
Change-Id: I39e5faa4ac22d915f06125aac36ced11607b94a3
Brad Jorsch [Wed, 26 Dec 2012 16:55:49 +0000 (11:55 -0500)]
Fix ForeignAPIRepo::fileExistsBatch()
ForeignAPIRepo::fileExistsBatch() is extremely broken:
* If passed an array with numeric keys (e.g. as called from
FileRepo::fileExists()), it will cache the fact that the item at index
0 from the first call was found or not found, no matter what the
actual file being queried is!
* But it doesn't even use its cache correctly. Rather than checking the
value cached, it just assumes anything that was cached exists.
* It checks whether the page in the remote repository has a description
page, not whether a file actually exists for the title.
* But it doesn't do that right either, so in the vast majority of cases
it will decide everything exists.
All of this is offset by the fact that I can't find any code path that
would actually call this function.
Change-Id: I421e8892b642f5267b8dd755183ebd711db83152