Reedy [Fri, 16 Nov 2012 23:20:58 +0000 (23:20 +0000)]
Output special page name before attempting to run function
Related to bug 42210, we don't have a blank line, not knowing what it's actually running
QueryPage updates do this already!
Change-Id: I18eb0fb4c316744d92a81f5e64e201508b27eb27
MarkTraceur [Fri, 16 Nov 2012 21:54:25 +0000 (21:54 +0000)]
Merge "(bug 37158) display personal menu on top of page tabs on Vector"
umherirrender [Fri, 16 Nov 2012 20:47:01 +0000 (21:47 +0100)]
Unset DirectoryIterator in the FileBackend
This is necessory to close all handlers on the directory and than it is
possible to rmdir the directory without a 'permission denied' on a
windows machine.
Corresponding unit test failure:
1) FileBackendTest::testRecursiveClean
Dir mwstore://localtesting/unittest-cont1/e/a no longer exists
(FSFileBackend).
Failed asserting that true matches expected false.
Change-Id: Ide58c6d0ec101bf0ddcd6b7414c51075fc28825b
Translation updater bot [Fri, 16 Nov 2012 20:43:16 +0000 (20:43 +0000)]
Merge "Localisation updates from translatewiki.net."
Translation updater bot [Fri, 16 Nov 2012 20:20:48 +0000 (20:20 +0000)]
Localisation updates from translatewiki.net.
Change-Id: Iaa7b07c4e450b6d6ddbc53b9c095c8f16409f59c
Aaron Schulz [Fri, 16 Nov 2012 20:20:28 +0000 (20:20 +0000)]
Merge "[FileBackend] Stat caching improvements."
Daniel Kinzler [Fri, 16 Nov 2012 20:17:42 +0000 (20:17 +0000)]
Merge "(bug 32368) Add ParserCloned hook"
Aaron Schulz [Fri, 16 Nov 2012 20:02:42 +0000 (12:02 -0800)]
[FileBackend] Stat caching improvements.
Callers often tend to end up calling getFileStat(), at least indirectly,
or in various successive function on the same path. This created RTTs
when the file didn't exist since negatives were not cached. This change
does the following:
* Cache definitive negatives (404s) in the process cache.
Nothing is cached on failure (like network problems).
* Ignore process cache entries after a brief time period
so long running scripts do not have overly stale entries.
Change-Id: I356bd9f48281e3c7e7a273778b2aca59c521a0c7
IAlex [Fri, 16 Nov 2012 19:39:07 +0000 (19:39 +0000)]
Merge "(bug 42173) Standardize categories special pages output"
IAlex [Fri, 16 Nov 2012 19:34:21 +0000 (19:34 +0000)]
Merge "Parser parse requires title, make a caller require valid title too"
Niklas Laxström [Thu, 15 Nov 2012 15:25:09 +0000 (15:25 +0000)]
Parser parse requires title, make a caller require valid title too
Change-Id: I8a9cd18c5aa27493a0945765f80136cd0cc4f218
Krinkle [Fri, 16 Nov 2012 17:31:01 +0000 (17:31 +0000)]
Merge "(bug 40850) jquery.byteLimit: Always update prevSafeVal."
MatmaRex [Thu, 15 Nov 2012 12:23:44 +0000 (13:23 +0100)]
(bug 40850) jquery.byteLimit: Always update prevSafeVal.
Not doing this sometimes caused trimValForByteLength to compare
the new value to an empty string instead of the old value,
resulting in trimming always at the end instead of at the position
of insertion.
Change-Id: I2e46961efa4f82732d577f7e5f98fc80719c88bb
Subramanya Sastry [Fri, 16 Nov 2012 17:09:20 +0000 (17:09 +0000)]
Merge "Add tests for empty attribute values in tags and table syntax"
Gabriel Wicke [Fri, 16 Nov 2012 01:12:05 +0000 (17:12 -0800)]
Add tests for empty attribute values in tags and table syntax
The PHP parser implements this very inconsistently and not in line with the
HTML5 spec. We follow the HTML5 spec in Parsoid, so mark these tests as
disabled for the PHP parser and add FIXMEs.
Change-Id: Iee5d95f3ee5fcb596282097c52f729d33ccb1474
Anomie [Fri, 16 Nov 2012 16:56:34 +0000 (16:56 +0000)]
Merge "Regression test for detecting edit conflicts."
Brad Jorsch [Thu, 15 Nov 2012 00:05:24 +0000 (16:05 -0800)]
(bug 32368) Add ParserCloned hook
We store various bits of data as "expando" properties on the Parser
object, to pass information from one stage of the parser to another. If
the parser is cloned, however, we can run into trouble because two
different Parser objects are now manipulating the same extension data
structure; this often shows up when ParserClearState is called on one
clone and clears the state of the other as well.
Since a deep clone might be too expensive and still might be wrong in
some cases, it seems most useful to simply provide a ParserCloned hook
so extensions can just do The Right Thing.
Change-Id: Ieec65c908d71e89b9a66f83b9a626f842aadacbb
Hashar [Fri, 16 Nov 2012 15:07:25 +0000 (15:07 +0000)]
Merge "Let TextContentTest extends MediaWikiLangTestCase"
Hashar [Fri, 16 Nov 2012 13:23:03 +0000 (13:23 +0000)]
Merge "SearchEngineTest.php must extends MediaWikiLangTestCase"
Dereckson [Fri, 16 Nov 2012 01:01:36 +0000 (02:01 +0100)]
(bug 42173) Standardize categories special pages output
This change removes namespace prefix from [[Special:UncategorizedCategories]],
so the output is consistent with the other categories pages.
Change-Id: I033e570754ca81be900a1f5482b8429583ab9172
Siebrand [Fri, 16 Nov 2012 10:27:57 +0000 (10:27 +0000)]
Merge "Fix typo (seams => seems)"
Ori Livneh [Fri, 16 Nov 2012 10:21:55 +0000 (02:21 -0800)]
Fix typo (seams => seems)
Change-Id: Ied64fa71b9da9762ee6bad7a4bae5a7082258f05
IAlex [Fri, 16 Nov 2012 08:57:14 +0000 (08:57 +0000)]
Merge "Fix prop=userid in list=protectedtitles"
Demon [Fri, 16 Nov 2012 05:21:15 +0000 (05:21 +0000)]
Merge "drop orm_test table on teardown"
Aaron Schulz [Wed, 14 Nov 2012 19:40:36 +0000 (11:40 -0800)]
[Database] Fixed case where trx idle callbacks might be lost.
* (bug 41656) LoadBalancer always commits if there are callbacks pending.
This works by checking a new DatabaseBase::writesOrCallbacksPending() function.
* Made transaction idle callbacks run in autocommit mode. Generally callers already
want autocommit mode or do quick begin()/commit() calls anyway. The docs already
make stat that callbacks should close any connections they start, but this makes
it harder for people to forget about implicit transactions. Since the callbacks
often may happen in commitMasterChanges(), if transactions are left open, they
might not get committed by anything afterwards.
* Added sanity exceptions and warnings if callbacks get lost for some reason.
* Renamed trxIdleCallbacks -> mTrxIdleCallbacks for consistency.
Change-Id: I00e1e0fcdd7deeee1fbac6d0f295160479cb8962
Aaron Schulz [Fri, 16 Nov 2012 01:05:26 +0000 (17:05 -0800)]
Fixed uncaught thumbName() exceptions and use $_GET.
* Using $_REQUEST is bad since cookie data is also in there.
* Also made a few tiny code cleanups.
Change-Id: Ib7064b95a2788484d05cf457034fe1d5be298bc4
Gabriel Wicke [Fri, 16 Nov 2012 00:02:15 +0000 (16:02 -0800)]
Add test for table row followed by newlines and table heading
Change-Id: I208f5464523ef3e76cd9ddc5c1198aa231bca9b8
Translation updater bot [Thu, 15 Nov 2012 20:32:17 +0000 (20:32 +0000)]
Localisation updates from translatewiki.net.
Change-Id: I620f7ee98965b8d07caeeaf19d7db40efcfc7975
daniel [Mon, 12 Nov 2012 15:39:29 +0000 (16:39 +0100)]
Regression test for detecting edit conflicts.
Change-Id: I9f8f3850ff0d6d51f173f5be73ffcb3b64886235
IAlex [Thu, 15 Nov 2012 10:24:11 +0000 (10:24 +0000)]
Merge "Made SSL validation in Curl HTTP requests the default."
IAlex [Thu, 15 Nov 2012 10:17:23 +0000 (10:17 +0000)]
Merge "Bug 42039 - Fix some file-related issues in the distribution Minor issues."
IAlex [Thu, 15 Nov 2012 10:14:43 +0000 (10:14 +0000)]
Merge "Uniform $rtl usage in Arabic-script Messages*.php"
IAlex [Thu, 15 Nov 2012 10:10:00 +0000 (10:10 +0000)]
Merge "Add 'contentmodel' to resultproperties in prop=revisions"
IAlex [Thu, 15 Nov 2012 10:05:28 +0000 (10:05 +0000)]
Merge "When returning rev content, always include model."
IAlex [Thu, 15 Nov 2012 09:59:23 +0000 (09:59 +0000)]
Merge "Fixed action=query list=blocks for IPv6 addresses"
Ori Livneh [Thu, 15 Nov 2012 09:47:54 +0000 (01:47 -0800)]
Fix typo: exipration => expiration
Change-Id: I73d19cfeed0596b66e16e4c383b3658594c211e3
Aaron Schulz [Thu, 8 Nov 2012 17:50:00 +0000 (09:50 -0800)]
Moved "create" function definitions before "store" for consistency.
Change-Id: Ic44470d7a178cb8dcd8f6a9343f961c26fb92988
Aaron Schulz [Thu, 15 Nov 2012 08:49:48 +0000 (00:49 -0800)]
[JobQueue] Fixed de-listing updates to pending job cache.
Change-Id: Ied0f2962fcbcde62a041592933c667c861f54715
Gabriel Wicke [Thu, 15 Nov 2012 01:04:30 +0000 (17:04 -0800)]
Add a test for indent pres with content wrapped in inline element
A line of content fully wrapped in an inline element was an issue in Parsoid
round-tripping, so make sure this is detected in the future.
Parsoid fix: https://gerrit.wikimedia.org/r/#/c/33496/
Change-Id: I2497c90174fac69e9c4dc1cc898844e94e604b1b
Brad Jorsch [Wed, 14 Nov 2012 22:51:06 +0000 (14:51 -0800)]
Fix intermittent QUnit failure
Occasionally the test 'mediawiki: mw.loader.implement( styles={ "url": {
<media>: [url, ..] } } )' will fail because it only ran 5 of 7 expected
assertions, while at the same time a later test will fail because it had
two extra assertions.
The problem is that the test is performing two external CSS loads, but
calls QUnit.start() after the ''first'' one is loaded. If the second
doesn't manage to get loaded at close to the same time, QUnit moves on
to the next test, and then when the second external CSS file finally
does load it will get counted towards whichever test is running at that
time. This can be reliably reproduced by arranging for
tests/qunit/data/styleTest.css.php to take longer for one than the other
of these two external CSS loads.
The solution is to wait until *both* of those external CSS loads to
complete (or time out) before calling QUnit.start().
Change-Id: Id695e95733b4c3f58234d9688c6b6e1b9ba591cc
Amire80 [Wed, 14 Nov 2012 21:25:05 +0000 (21:25 +0000)]
Merge "(bug 2865) Add lang attribute to firstHeading of CologneBlue"
Translation updater bot [Wed, 14 Nov 2012 21:00:37 +0000 (21:00 +0000)]
Merge "Localisation updates from translatewiki.net."
Derk-Jan Hartman [Wed, 14 Nov 2012 20:59:48 +0000 (21:59 +0100)]
(bug 2865) Add lang attribute to firstHeading of CologneBlue
This is a follow up to I55dd392d
Change-Id: I20bd5b6aae6a26f149a4a822aefdea860f148093
Anomie [Wed, 14 Nov 2012 20:59:03 +0000 (20:59 +0000)]
Merge "(bug 42097) $.tablesorter fix when wgDefaultDateFormat is unrecognized"
MatmaRex [Wed, 14 Nov 2012 20:25:43 +0000 (21:25 +0100)]
(bug 42097) $.tablesorter fix when wgDefaultDateFormat is unrecognized
The 'date' parser assumed all wikis use wgDefaultDateFormat that is
either 'mdy' or 'dmy', but this is not the case; for example
fi.wikipedia uses 'fi normal'. Therefore the correct value was not set,
a string was not changed to an array, and a loop that assumed an array
would be present hanged forever.
Change-Id: Iba17bc6b44d6a2fff491b7462d7a5457f85d19c5
Amir E. Aharoni [Mon, 12 Nov 2012 17:45:22 +0000 (23:15 +0530)]
(bug 2865) Add lang attribute to firstHeading
Adds a lang attribute to <h1 id="firstHeading">.
Interface language is applied to <html> and content language is
applied to <div id="mw-content-text">. This leaves firstHeading
with the interface language, which is wrong in general. It
also causes incorrect display of the title for languages that
have custom line-height.
This patch is based on code written by Munirathnam M at
the Bangalore DevCamp (he couldn't submit it because of network
issues). It looks OK to me, but it's my first time doing anything
with $this->data in a skin, so please correct me if I'm wrong.
Change-Id: I55dd392dbf0d4f768344b66e6ceb3a023b1d5c9b
Translation updater bot [Wed, 14 Nov 2012 20:33:44 +0000 (20:33 +0000)]
Localisation updates from translatewiki.net.
Change-Id: Ic7c0ea324127520a1e62d0ceeda1003990d318b3
Brad Jorsch [Fri, 9 Nov 2012 19:48:46 +0000 (11:48 -0800)]
(bug 41886) jquery.tablesorter should not explode rowspans until sorted
jquery.tablesorter should not explode rowspans until the table is
actually sorted.
Also adds a testcase to verify this behavior, and updates another test
case that was depending on the broken behavior.
Change-Id: Ibe4cc7e92de6cc8e370c5912c9eda062b466c538
daniel [Wed, 14 Nov 2012 11:17:13 +0000 (12:17 +0100)]
When returning rev content, always include model.
query=revisions can be used with rvprop=content to retrieve the
content of revisions. The contentmodel should always be included
with the content (just like the contenformat is), so the client
is able to interpret the structure contained in the content blob.
Change-Id: I67cf48f905ff83a86992e1a54f7ad0feaf2b2c94
Aaron Schulz [Wed, 14 Nov 2012 07:00:09 +0000 (23:00 -0800)]
[JobQueue] Added missing delete query.
Change-Id: I8f900f5d7af3f42e348ac733cdf8deff37d4d055
Niklas Laxström [Wed, 14 Nov 2012 07:43:02 +0000 (07:43 +0000)]
Typofix
Change-Id: I50bf84c3d7f75f6d2a0f3d4f8e86622486f2be28
Aaron Schulz [Wed, 24 Oct 2012 00:29:54 +0000 (17:29 -0700)]
[JobQueue] Added support for retrying jobs when runners die.
* This adds a new 'claimTTL' setting which recycles jobs into the queue
if they have been claimed for longer than that duration (in seconds).
* This also purges jobs for such cases when job retries are not enabled.
* This adds a new job_attempts column and adds an index to improve the
query and semantics for FIFO ordered queues.
Change-Id: Idb6217a694d86a4d6fc881768deed424628f015d
Anomie [Tue, 13 Nov 2012 23:32:27 +0000 (23:32 +0000)]
Merge "Separated validation login in SpecialBlock."
Anomie [Tue, 13 Nov 2012 23:28:59 +0000 (23:28 +0000)]
Merge "(bug 7851) Implement mediawiki.page.patrol.ajax"
Anomie [Tue, 13 Nov 2012 23:08:16 +0000 (23:08 +0000)]
Merge "Set default type attribute for button html elements"
Gabriel Wicke [Tue, 13 Nov 2012 21:43:43 +0000 (13:43 -0800)]
Add a test covering template targets containing wiki links
Change-Id: I0db7b0387dde5a9b450af486a66f451377de1d7e
Marius Hoch [Wed, 3 Oct 2012 01:33:55 +0000 (03:33 +0200)]
(bug 7851) Implement mediawiki.page.patrol.ajax
Implement AJAX patrolling with the new mediawiki.page.patrol.ajax
module, which makes use of the API via mediawiki.api.
During the patrol process a spinner (created by jquery.spinner)
shows up and after it a suitable message gets shown via
mediawiki.notify.
Depending on whether we had success or not the link then turns up
again or the brackets completely disappear just like on a normal page view.
On top of adding the module, I've changed the following:
- Added the patrol token to the ResourceLoaderUserTokensModule.
- Registered messages 'markedaspatrollednotify' and
'markedaspatrollederrornotify'.
Change-Id: I472357566dda0ab572c20e2e4b87508b0f2f4c73
GWicke [Tue, 13 Nov 2012 21:29:16 +0000 (21:29 +0000)]
Merge "(bug 41989) tests/parserTests.php missing classes"
Antoine Musso [Tue, 13 Nov 2012 20:41:40 +0000 (21:41 +0100)]
(bug 41989) tests/parserTests.php missing classes
It has been broken since I0a307f23 /
7c6c05a which moved the parser test
classes from the main autoloader to the one dedicated to test.
Workaround that by simply requiring the two .inc file we need.
Change-Id: Ib865e4e84969d633ad9173f0824322078793b8b4
Translation updater bot [Tue, 13 Nov 2012 20:52:49 +0000 (20:52 +0000)]
Localisation updates from translatewiki.net.
Change-Id: I0b01fe52457790ea132defbf45a31b8cb1f5b199
Amir E. Aharoni [Tue, 13 Nov 2012 13:40:31 +0000 (15:40 +0200)]
Uniform $rtl usage in Arabic-script Messages*.php
Also fix the name of aeb (Tunisian Arabic).
Change-Id: I63c414c090d67e61b16df234eeb2043733203e1a
Ori Livneh [Tue, 13 Nov 2012 04:20:07 +0000 (04:20 +0000)]
Merge "Fix typo (ResourceLodaer => ResourceLoader)"
Ori Livneh [Tue, 13 Nov 2012 04:18:59 +0000 (20:18 -0800)]
Fix typo (ResourceLodaer => ResourceLoader)
Change-Id: Ie349853f3fbcd7a9eb038d5988497f115684ebeb
Timo Tijhof [Tue, 13 Nov 2012 03:10:13 +0000 (04:10 +0100)]
Release-notes: Fixup for I70656b41bd2.
Change-Id: If786a387101a7e801c901f00bca8c4a875ceb56d
Translation updater bot [Mon, 12 Nov 2012 20:52:16 +0000 (20:52 +0000)]
Localisation updates from translatewiki.net.
Change-Id: I6b51ce2eb452830792d858e143ca06a017648699
daniel [Tue, 6 Nov 2012 13:05:30 +0000 (14:05 +0100)]
(Bug 41706) preload to convert content as needed.
Use the new Content::convert function to convert preloaded content
if the page being edited uses a different content model. This allows
e.g. a page like User:Foo/bar to be preloaded when creating
User:Foo/common.css. This currently fails, because it is not possible
to use wikitext to pre-fill a CSS page.
Change-Id: I70656b41bd203e5722528e889a242fa54cbfbf62
Brad Jorsch [Mon, 12 Nov 2012 05:42:26 +0000 (21:42 -0800)]
(bug 41990) Fix regression in API edit of redirect
Before ContentHandler, an API edit passing a redirect for title and
redirect=true and omitting basetimestamp would use the last revision
timestamp of the target page. When ContentHandler was merged, this was
accidentally changed to use the last revision timestamp of the redirect
page, which was likely much earlier than the target page's last revision
and so causes an edit conflict in almost all cases.
Some scripts took advantage of this along with appendtext to add notices
to users' talk pages, so restore that old behavior. We'll also adjust
the contentmodel/contentformat detection added by ContentHandler to use
the model and format of the target page rather than the redirect in this
case, as that seems more likely to be less wrong.
Change-Id: If0c674e26a4deb54ec14f0bf45418d666a397347
MarkAHershberger [Mon, 12 Nov 2012 14:23:51 +0000 (09:23 -0500)]
Bug 42039 - Fix some file-related issues in the distribution
Minor issues.
Tested the removal of
maintenance/postgres/archives/patch-ipb_address_unique.sql in a new
installation but haven't tested it on an upgrade.
Change-Id: I58aa11c5acab5de427cbc000e6786a208fc6b26f
daniel [Mon, 12 Nov 2012 13:37:32 +0000 (14:37 +0100)]
fix conflict marker in comment
Change-Id: I23e9593890ab5899e10d8eb61f63f38fb98072a7
Brad Jorsch [Mon, 12 Nov 2012 04:53:22 +0000 (20:53 -0800)]
(bug 42036) Account creation email link needs protocol
In other words, use $wgCanonicalServer instead of $wgServer.
Change-Id: I110db4d27ce935d732e16830951af0f349e99e99
TheDJ [Sun, 11 Nov 2012 21:06:49 +0000 (21:06 +0000)]
Merge "Fix E_STRICT in ForeignAPIRepo"
Translation updater bot [Sun, 11 Nov 2012 21:06:35 +0000 (21:06 +0000)]
Merge "Localisation updates from translatewiki.net."
Translation updater bot [Sun, 11 Nov 2012 20:29:48 +0000 (20:29 +0000)]
Localisation updates from translatewiki.net.
Change-Id: I6b4b6352c7debed2fab97b7f72f960613f41c496
IAlex [Sun, 11 Nov 2012 20:23:41 +0000 (20:23 +0000)]
Merge "Remove unneeded third param"
IAlex [Sun, 11 Nov 2012 20:18:46 +0000 (20:18 +0000)]
Merge "Set wgLanguageCode/wgContLang for backup_PageTest.php"
IAlex [Sun, 11 Nov 2012 20:15:16 +0000 (20:15 +0000)]
Merge "fix @depends"
Demon [Sun, 11 Nov 2012 19:16:38 +0000 (19:16 +0000)]
Merge changes I4ac3bc14,I470acb8b
* changes:
make addPortletLink work with CologneBlue
(bug 18338) WAI-ARIA landmark roles in most used skins
umherirrender [Sun, 11 Nov 2012 07:13:21 +0000 (08:13 +0100)]
SearchEngineTest.php must extends MediaWikiLangTestCase
There is namespace handling, which fails on $wgLanguageCode = 'de';
Change-Id: I6caa403e5e5ba55c0666d989762aaebe8ced878f
umherirrender [Sun, 11 Nov 2012 07:08:45 +0000 (08:08 +0100)]
Let TextContentTest extends MediaWikiLangTestCase
There is signature expansion, which fails on $wgLanguageCode = 'de';
Change-Id: I827693a9f872219b983c62f4d3483e67ed9e8a2f
Nikerabbit [Sun, 11 Nov 2012 08:14:35 +0000 (08:14 +0000)]
Merge "Make sure that SQLite uses no prefix"
Amir E. Aharoni [Sun, 11 Nov 2012 04:39:37 +0000 (10:09 +0530)]
Cleanup Hebrew grammar code a little.
Added comments and changed == to ===.
Change-Id: I3124bfd7341b1b3b46bb3e5215001d6967845f48
Marius Hoch [Sun, 11 Nov 2012 02:25:17 +0000 (03:25 +0100)]
Fixed action=query list=blocks for IPv6 addresses
Queries like the following return wrong rangestart and rangeend
data cause the IP addresses are formated with only IPv4 in mind.
As IP::formatHex is able to format both I've switched over to
that.
Example:
http://en.wikipedia.org/w/api.php?format=xml&action=query&list=blocks&bkprop=range&bkip=2002%3AB752%3AD225%3A0%3A0%3A0%3A0%3A1
Change-Id: I6138240bb03bbf5af61851d3e96354a050e56fed
Krinkle [Sat, 10 Nov 2012 20:53:02 +0000 (20:53 +0000)]
Merge "test classes autoloading path was broken"
Antoine Musso [Sat, 10 Nov 2012 20:39:54 +0000 (21:39 +0100)]
test classes autoloading path was broken
Since
7c6c05a, the ORMTableTest and SeleniumTestConstants classes points
to a wrong path. The reason is the variable name has not been renamed
for them.
Change-Id: I430031d0d00a8f0da047184285d1797a145bd389
Translation updater bot [Sat, 10 Nov 2012 20:33:18 +0000 (20:33 +0000)]
Localisation updates from translatewiki.net.
Change-Id: Ic5565a4999b2212f6be13506f413ac918c6e8fc0
Ori Livneh [Sat, 10 Nov 2012 19:49:36 +0000 (11:49 -0800)]
Removed unused var from jquery.badge.js
Change-Id: Ia3313ee272218d6467675e8e855710a9365d3fa2
umherirrender [Sat, 10 Nov 2012 19:15:03 +0000 (20:15 +0100)]
fix @depends
No need for the parentheses there
Change-Id: Id2ba72a7fb7bdd592de0642468994c9dab29adcd
umherirrender [Sat, 10 Nov 2012 16:42:03 +0000 (17:42 +0100)]
drop orm_test table on teardown
No need to keep this table after the test
Change-Id: Id6814ceca063294da0761caa9a0d9975118dd4f1
umherirrender [Sat, 10 Nov 2012 18:50:13 +0000 (19:50 +0100)]
Set wgLanguageCode/wgContLang for backup_PageTest.php
There is namespace handling, which fails on $wgLanguageCode = 'de';
Change-Id: I99a4a1837e5db5eb22cb879898da3ac37439264d
Timo Tijhof [Wed, 7 Nov 2012 01:27:30 +0000 (02:27 +0100)]
(bug 41833) Test: Add AutoLoaderTest.
* Move scattered pieces from tests/* in main AutoLoader.php
into tests/TestsAutoLoader.php.
Verified with:
'ack -Q i --ignore-dir tests/phpunit/ <classname>'
on mediawiki/core that these classes are not used outside
tests/phpunit/.
* Moved entry for maintenance/backup.inc to the main AutoLoader.
* Refactored assertion logic in maintenance/checkAutoLoader.php
into a public static method used in it's execute method
and in the (new) AutoLoaderTest suite.
* The new test was immediately failing, added missing classes
and removed old ones that don't exist.
And CheckAutoLoader itself, so that it can actually be used
in AutoLoaderTest.php
* Per discussion on Gerrit, moved the logic into the unit test
instead of refactoring the maintenance script, we no longer
need the maintenance script.
* Fixed the regex to also detect abstract, final and interface.
The test was failing badly, claiming many classes did not
exist.
* Improved the logic to also catch entries in the AutoLoader
configuration for inexisting classes in existing files.
So far it only catched entries with wrong files and missing
entries for classes in known files.
An entry like "BlablaSomethingHere => includes/Action.php"
did not emit any kind of warning. The refactored logic
builds a reverse index and uses a simple assertEquals to
find any inconsistencies (whatever the cause).
Change-Id: I0a307f23175d52345180cdfc7c2d5e172536be1b
Daniel Kinzler [Sat, 10 Nov 2012 16:18:12 +0000 (17:18 +0100)]
avoid STDERR diff3 messages when using wfMerge()
diff3 issues a warning to stderr if any of the files does not end with a
newline character.
TextContent::preSaveTransform() does normalize revision text by simply
calling trim(). Thus to avoid a diff3 error we simply apply the same
normalization and add a newline to please the command.
Change-Id: I7baa3df95dd70cbc865b2491ccc791e60f8b9e6e
Hashar [Sat, 10 Nov 2012 16:44:44 +0000 (16:44 +0000)]
Merge "(Bug 41352) Provide tests for edit conflicts."
daniel [Sat, 10 Nov 2012 16:24:48 +0000 (17:24 +0100)]
(Bug 41352) Provide tests for edit conflicts.
This provides several test cases for EditPage, including some for
detecting and resolving edit conflicts. This is in preparation for
finding and eliminating the root cause of bug 41352.
Change-Id: I8f23c6535f7b3ee6462d9c5a19dd06070e77f9d1
umherirrender [Sat, 10 Nov 2012 16:23:32 +0000 (17:23 +0100)]
Remove unneeded third param
the 'messages' are already part of 'preload', so there is no need for
that. Last changes there: r101507/r102304
Change-Id: I6c9daa3c46bd7f1db851cc4620ecf59f81e6f9aa
Reedy [Sat, 10 Nov 2012 16:15:56 +0000 (16:15 +0000)]
Merge "Follow-up Ib2b8dfb8 (
803abc7): fix a typo in a variable name"
Alexandre Emsenhuber [Sat, 10 Nov 2012 16:12:45 +0000 (17:12 +0100)]
Follow-up Ib2b8dfb8 (
803abc7): fix a typo in a variable name
$mRetypePass should exist, the correct name is $mRetype.
Change-Id: I0bd65951f42a244a8810087f900b4a3c82358405
IAlex [Sat, 10 Nov 2012 16:11:33 +0000 (16:11 +0000)]
Merge "Explicitly define member variables"
Alexandre Emsenhuber [Sat, 10 Nov 2012 15:29:56 +0000 (16:29 +0100)]
* (bug 41966) Follow-up I551a90b0 (
c2172d4): fix fatal error in PoolWorkArticleView::doWork()
The error is Fatal error: Call to a member function getParserOutput() on a non-object in includes/WikiPage.php on line 3363.
The problem is that if the fifth parameter passed to PoolWorkArticleView::__construct() is not given or is null, which is now
the case since the change mentioned on the first line of this commit summary, the content is fetched from Page::getContent().
When the Page object is a WikiPage instance there is no problem, but when this is an Article instance, Article::getContent()
actually returns the content as *string* and not as object, like it did before the Wikidata branch merge.
Worked arround this by passing the WikiPage object to PoolWorkArticleView from Article::view().
Also removed non-existing sixth parameter to PoolWorkArticleView::__construct().
Change-Id: I537862f519542de4f034be29bba4b75898382f59
Reedy [Sat, 10 Nov 2012 15:05:02 +0000 (15:05 +0000)]
Explicitly define member variables
Change-Id: Ib2b8dfb84570557f1aeab474f4939e3a657c5732
daniel [Mon, 29 Oct 2012 17:38:11 +0000 (18:38 +0100)]
(Bug 41244) Gracefully handle failure to load text blob.
This change fixes fatal errors ocurring when Revision::loadText
returns false because of a failure to load the text blob.
Revision::getContent() should simply return null in such a case.
Change-Id: I1e13de14ff15b124b5a2e07155d368595a16fda7