[MediaWiki-commits] [Gerrit] Remove error suppression in Exif::isRational - change (mediawiki/core)

2013-03-17 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/54215 Change subject: Remove error suppression in Exif::isRational .. Remove error suppression in Exif::isRational preg_match throws warning,

[MediaWiki-commits] [Gerrit] Remove spaces between parentheses in function calls - change (mediawiki/core)

2013-03-17 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/54216 Change subject: Remove spaces between parentheses in function calls .. Remove spaces between parentheses in function calls Change-Id: I

[MediaWiki-commits] [Gerrit] Fix type of list=tags&tgcontinue - change (mediawiki/core)

2013-03-17 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/54217 Change subject: Fix type of list=tags&tgcontinue .. Fix type of list=tags&tgcontinue Was already interpretted as string before, but wit

[MediaWiki-commits] [Gerrit] Remove spaces in function signature - change (mediawiki/core)

2013-03-18 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/54517 Change subject: Remove spaces in function signature .. Remove spaces in function signature Change-Id: I45aea7a7af88cd913b2f485913620a8a

[MediaWiki-commits] [Gerrit] Remove (edit) from Special:Tags for non-editinterface users - change (mediawiki/core)

2013-03-19 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/54715 Change subject: Remove (edit) from Special:Tags for non-editinterface users .. Remove (edit) from Special:Tags for non-editinterface use

[MediaWiki-commits] [Gerrit] ContentHandlerTest: Use dataProvider - change (mediawiki/core)

2013-03-20 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/54879 Change subject: ContentHandlerTest: Use dataProvider .. ContentHandlerTest: Use dataProvider Changing a global 3 times in a test is bad

[MediaWiki-commits] [Gerrit] SideBarTest.php: Use dataProvider/use setMwGlobals - change (mediawiki/core)

2013-03-20 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/54886 Change subject: SideBarTest.php: Use dataProvider/use setMwGlobals .. SideBarTest.php: Use dataProvider/use setMwGlobals One test metho

[MediaWiki-commits] [Gerrit] MagicVariableTest.php: Use dataProvider - change (mediawiki/core)

2013-03-20 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/54892 Change subject: MagicVariableTest.php: Use dataProvider .. MagicVariableTest.php: Use dataProvider Setting a global 3 times in one test

[MediaWiki-commits] [Gerrit] TimeAdjustTest: Use dataProvider - change (mediawiki/core)

2013-03-21 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/55095 Change subject: TimeAdjustTest: Use dataProvider .. TimeAdjustTest: Use dataProvider Also removes setting of wgContLang/wgLanguageCode,

[MediaWiki-commits] [Gerrit] MessageTest: Split test and use setMwGlobals - change (mediawiki/core)

2013-03-21 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/55118 Change subject: MessageTest: Split test and use setMwGlobals .. MessageTest: Split test and use setMwGlobals Change-Id: I9d7f16ed519622

[MediaWiki-commits] [Gerrit] Tests: Use more setMwGlobals - change (mediawiki/core)

2013-03-21 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/55121 Change subject: Tests: Use more setMwGlobals .. Tests: Use more setMwGlobals Change some tests to use setMwGlobals to have restoring of

[MediaWiki-commits] [Gerrit] Tests: Fix case of some methods - change (mediawiki/core)

2013-03-22 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/55272 Change subject: Tests: Fix case of some methods .. Tests: Fix case of some methods Change-Id: Iab247f805410f2e9157d325edd80880538a3e54c

[MediaWiki-commits] [Gerrit] Installer: Check if /extensions is readable and a directory - change (mediawiki/core)

2013-03-22 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/55289 Change subject: Installer: Check if /extensions is readable and a directory .. Installer: Check if /extensions is readable and a directo

[MediaWiki-commits] [Gerrit] DefaultSettings: Remove some empty lines - change (mediawiki/core)

2013-03-22 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/55303 Change subject: DefaultSettings: Remove some empty lines .. DefaultSettings: Remove some empty lines Add a missing comment - text taken

[MediaWiki-commits] [Gerrit] Pass function name to insert() in replace() in Database class - change (mediawiki/core)

2013-03-23 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/55460 Change subject: Pass function name to insert() in replace() in Database class .. Pass function name to insert() in replace() in Database

[MediaWiki-commits] [Gerrit] Improve help text for list=alllinks and alltransclusions - change (mediawiki/core)

2013-06-21 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/69950 Change subject: Improve help text for list=alllinks and alltransclusions .. Improve help text for list=alllinks and alltransclusions Th

[MediaWiki-commits] [Gerrit] Capitalize URL in api help - change (mediawiki/core)

2013-06-21 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/69953 Change subject: Capitalize URL in api help .. Capitalize URL in api help As part of bug 34965 Change-Id: Ia1df9748b5de33f6ef5088103ccc

[MediaWiki-commits] [Gerrit] date_default_timezone_set unnneeded in Parser::getVariableValue - change (mediawiki/core)

2013-06-22 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/69971 Change subject: date_default_timezone_set unnneeded in Parser::getVariableValue .. date_default_timezone_set unnneeded in Parser::getVar

[MediaWiki-commits] [Gerrit] Added wf[Local]DateTime to create DateTime objects - change (mediawiki/core)

2013-06-28 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/71121 Change subject: Added wf[Local]DateTime to create DateTime objects .. Added wf[Local]DateTime to create DateTime objects This avoids us

[MediaWiki-commits] [Gerrit] Use wfAppendQuery to append a query - change (mediawiki/core)

2013-04-11 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/58728 Change subject: Use wfAppendQuery to append a query .. Use wfAppendQuery to append a query Also pass a array to wfAppendQuery, which th

[MediaWiki-commits] [Gerrit] Use a combined const, instead of string concat in Exif.php - change (mediawiki/core)

2013-04-12 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/58903 Change subject: Use a combined const, instead of string concat in Exif.php .. Use a combined const, instead of string concat in Exif.php

[MediaWiki-commits] [Gerrit] Fixed spacing in skins folder - change (mediawiki/core)

2013-04-15 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/59180 Change subject: Fixed spacing in skins folder .. Fixed spacing in skins folder Added spaces inside function calls Added spaces before i

[MediaWiki-commits] [Gerrit] Add missing braces to for in wfGetCallerTest - change (mediawiki/core)

2013-04-17 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/59654 Change subject: Add missing braces to for in wfGetCallerTest .. Add missing braces to for in wfGetCallerTest Change-Id: I14b7abb90e2574

[MediaWiki-commits] [Gerrit] Pass array to Title::getLocalURL and friends instead of strings - change (mediawiki/core)

2013-04-18 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/59855 Change subject: Pass array to Title::getLocalURL and friends instead of strings .. Pass array to Title::getLocalURL and friends instead

[MediaWiki-commits] [Gerrit] Fixed spacing in api folder - change (mediawiki/core)

2013-04-19 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/60028 Change subject: Fixed spacing in api folder .. Fixed spacing in api folder Added spaces before if, foreach Added some braces for one li

[MediaWiki-commits] [Gerrit] Remove duplicate code from Setup.php - change (mediawiki/core)

2013-04-20 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/60107 Change subject: Remove duplicate code from Setup.php .. Remove duplicate code from Setup.php Change-Id: I6dadb12ec8410258d7c9c24e1d12ba

[MediaWiki-commits] [Gerrit] Fixed spacing in db/debug/diff/externalstore/objectcache folder - change (mediawiki/core)

2013-04-20 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/60129 Change subject: Fixed spacing in db/debug/diff/externalstore/objectcache folder .. Fixed spacing in db/debug/diff/externalstore/objectca

[MediaWiki-commits] [Gerrit] Add doc for SpecialPage::postText - change (mediawiki/core)

2013-04-20 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/60139 Change subject: Add doc for SpecialPage::postText .. Add doc for SpecialPage::postText A combined doc does not work Add some spaces to

[MediaWiki-commits] [Gerrit] Add a OneLineStatementSniff - change (mediawiki...codesniffer)

2013-04-21 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/60199 Change subject: Add a OneLineStatementSniff .. Add a OneLineStatementSniff This sniffs looks for curly braces around ifs and report mis

[MediaWiki-commits] [Gerrit] Use wfShellExec in MimeMagic - change (mediawiki/core)

2013-04-21 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/60201 Change subject: Use wfShellExec in MimeMagic .. Use wfShellExec in MimeMagic This avoids shell execution with backticks Change-Id: I37

[MediaWiki-commits] [Gerrit] $wgNamespaceRobotPolicies for dewiki: Add 101 and 829 - change (operations/mediawiki-config)

2013-04-23 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/60445 Change subject: $wgNamespaceRobotPolicies for dewiki: Add 101 and 829 .. $wgNamespaceRobotPolicies for dewiki: Add 101 and 829 dewiki h

[MediaWiki-commits] [Gerrit] NS_IMAGE/NS_IMAGE_TALK => NS_FILE/NS_FILE_TALK - change (operations/mediawiki-config)

2013-04-23 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/60447 Change subject: NS_IMAGE/NS_IMAGE_TALK => NS_FILE/NS_FILE_TALK .. NS_IMAGE/NS_IMAGE_TALK => NS_FILE/NS_FILE_TALK Change-Id: I32f93e1786

[MediaWiki-commits] [Gerrit] Use RecentChange::getTitle in ExternalChangesLine - change (mediawiki...Wikibase)

2013-04-23 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/60452 Change subject: Use RecentChange::getTitle in ExternalChangesLine .. Use RecentChange::getTitle in ExternalChangesLine Title::newFromTe

[MediaWiki-commits] [Gerrit] $wgFlaggedRevsNamespaces for dewiki: Add NS_MODULE - change (operations/mediawiki-config)

2013-05-09 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/62979 Change subject: $wgFlaggedRevsNamespaces for dewiki: Add NS_MODULE .. $wgFlaggedRevsNamespaces for dewiki: Add NS_MODULE dewiki already

[MediaWiki-commits] [Gerrit] Avoid Generic accessor in Category.php - change (mediawiki/core)

2013-05-09 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/62986 Change subject: Avoid Generic accessor in Category.php .. Avoid Generic accessor in Category.php This php magic is hard to read Change

[MediaWiki-commits] [Gerrit] echo is not a function - change (mediawiki/core)

2013-05-09 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/62988 Change subject: echo is not a function .. echo is not a function Removed parenthesis after echo Change-Id: Ia533aedf63b11d15dcc6a5cf75

[MediaWiki-commits] [Gerrit] Fixed spacing - change (mediawiki/core)

2013-05-09 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/62994 Change subject: Fixed spacing .. Fixed spacing Added spaces around some parenthesis Change-Id: If1e50e2a7b0046e91c0bbce8bf6641d8b2446a

[MediaWiki-commits] [Gerrit] print is not a function - change (mediawiki/core)

2013-05-09 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/62999 Change subject: print is not a function .. print is not a function Removed parenthesis after print Change-Id: I1343872de7aa7c64952a3d8

[MediaWiki-commits] [Gerrit] jQuery.byteLimit with callback can go into a endless loop - change (mediawiki/core)

2013-05-11 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/63282 Change subject: jQuery.byteLimit with callback can go into a endless loop .. jQuery.byteLimit with callback can go into a endless loop

[MediaWiki-commits] [Gerrit] Do not returning external title in ApiPageSet::getRedirectTa... - change (mediawiki/core)

2013-05-11 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/63288 Change subject: Do not returning external title in ApiPageSet::getRedirectTargets .. Do not returning external title in ApiPageSet::get

[MediaWiki-commits] [Gerrit] Set english transform tables in jquery.tablesorter.test - change (mediawiki/core)

2013-05-13 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/63482 Change subject: Set english transform tables in jquery.tablesorter.test .. Set english transform tables in jquery.tablesorter.test Havi

[MediaWiki-commits] [Gerrit] Fixed spacing and removed unneeded parenthesis - change (mediawiki/core)

2013-05-15 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/63874 Change subject: Fixed spacing and removed unneeded parenthesis .. Fixed spacing and removed unneeded parenthesis Added spaces after/bef

[MediaWiki-commits] [Gerrit] Replaced some function calls with local vars - change (mediawiki/core)

2013-05-15 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/63896 Change subject: Replaced some function calls with local vars .. Replaced some function calls with local vars Storing the [fix] return v

[MediaWiki-commits] [Gerrit] Fix badcontinue in possible errors of paraminfo - change (mediawiki/core)

2013-05-15 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/64024 Change subject: Fix badcontinue in possible errors of paraminfo .. Fix badcontinue in possible errors of paraminfo Badcontinue results

[MediaWiki-commits] [Gerrit] Add helpurls to some api modules - change (mediawiki/core)

2013-05-16 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/64026 Change subject: Add helpurls to some api modules .. Add helpurls to some api modules Change-Id: I41cc20ea5daf089e5fb26d1548d3b58cfe5f4d

[MediaWiki-commits] [Gerrit] Add RC_EXTERNAL to comment in tables.sql - change (mediawiki/core)

2013-05-17 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/64317 Change subject: Add RC_EXTERNAL to comment in tables.sql .. Add RC_EXTERNAL to comment in tables.sql Change-Id: If3156984e9ff447fec6644

[MediaWiki-commits] [Gerrit] Add page language to prop=info - change (mediawiki/core)

2013-05-18 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/64512 Change subject: Add page language to prop=info .. Add page language to prop=info This can be useful to determine the language of a page

[MediaWiki-commits] [Gerrit] Add non DBMS depending SQL tests for DatabaseBase - change (mediawiki/core)

2013-03-26 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/55934 Change subject: Add non DBMS depending SQL tests for DatabaseBase .. Add non DBMS depending SQL tests for DatabaseBase Created a Databa

[MediaWiki-commits] [Gerrit] Check return value of dir() in Skin.php - change (mediawiki/core)

2013-03-27 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/56109 Change subject: Check return value of dir() in Skin.php .. Check return value of dir() in Skin.php This avoids a fatal, when something

[MediaWiki-commits] [Gerrit] Handle invalid titles on ProtectedPages and ProtectedTitles - change (mediawiki/core)

2013-03-27 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/56125 Change subject: Handle invalid titles on ProtectedPages and ProtectedTitles .. Handle invalid titles on ProtectedPages and ProtectedTitl

[MediaWiki-commits] [Gerrit] API: Set uselang as allowed param in ApiMain and document it - change (mediawiki/core)

2013-03-28 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/56373 Change subject: API: Set uselang as allowed param in ApiMain and document it .. API: Set uselang as allowed param in ApiMain and documen

[MediaWiki-commits] [Gerrit] API param validation: Set strict mode for in_array - change (mediawiki/core)

2013-04-02 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/57038 Change subject: API param validation: Set strict mode for in_array .. API param validation: Set strict mode for in_array The in_array g

[MediaWiki-commits] [Gerrit] API: Add more generic possible errors - change (mediawiki/core)

2013-04-02 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/57039 Change subject: API: Add more generic possible errors .. API: Add more generic possible errors Added the possible errors for baduser, b

[MediaWiki-commits] [Gerrit] API: Add new type 'language' - change (mediawiki/core)

2013-04-02 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/57047 Change subject: API: Add new type 'language' .. API: Add new type 'language' Working the same way than 'namespace' by override the type

[MediaWiki-commits] [Gerrit] API: Add new hook 'APIGetPossibleErrors' - change (mediawiki/core)

2013-04-02 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/57103 Change subject: API: Add new hook 'APIGetPossibleErrors' .. API: Add new hook 'APIGetPossibleErrors' This allows extensions to modify t

[MediaWiki-commits] [Gerrit] Update limit/urlwidth param doc of prop=imageinfo - change (mediawiki/core)

2013-04-03 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/57268 Change subject: Update limit/urlwidth param doc of prop=imageinfo .. Update limit/urlwidth param doc of prop=imageinfo Follow up Id0ec6

[MediaWiki-commits] [Gerrit] Remove spaces around :: - change (mediawiki/core)

2013-04-03 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/57277 Change subject: Remove spaces around :: .. Remove spaces around :: Change-Id: I24fe5ad907bcf5235aa91b650785c571502456a1 --- M includes/

[MediaWiki-commits] [Gerrit] fixed php doc in ApiBase - change (mediawiki/core)

2013-04-05 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/57741 Change subject: fixed php doc in ApiBase .. fixed php doc in ApiBase Change-Id: I9c893de8c067be3e926f5de5f0b87ac47b81772d --- M include

[MediaWiki-commits] [Gerrit] Call ApiPageSet::getFinalParamDescription - change (mediawiki/core)

2013-04-05 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/57745 Change subject: Call ApiPageSet::getFinalParamDescription .. Call ApiPageSet::getFinalParamDescription This allows extensions to modify

[MediaWiki-commits] [Gerrit] Add nosummary, minor and watchthis to Special:MyPage and fri... - change (mediawiki/core)

2013-04-06 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/57852 Change subject: Add nosummary, minor and watchthis to Special:MyPage and friends .. Add nosummary, minor and watchthis to Special:MyPage

[MediaWiki-commits] [Gerrit] Add missing wfProfileOut before throwing an exception - change (mediawiki/core)

2013-04-07 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/57922 Change subject: Add missing wfProfileOut before throwing an exception .. Add missing wfProfileOut before throwing an exception Change-I

[MediaWiki-commits] [Gerrit] Add missing wfProfileOut to LocalisationCache - change (mediawiki/core)

2013-04-08 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/58108 Change subject: Add missing wfProfileOut to LocalisationCache .. Add missing wfProfileOut to LocalisationCache Follow up Ib94a8c18c4e27

[MediaWiki-commits] [Gerrit] Fix doc and deprecated warning of Title::fixUrlQueryArgs - change (mediawiki/core)

2013-04-08 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/58120 Change subject: Fix doc and deprecated warning of Title::fixUrlQueryArgs .. Fix doc and deprecated warning of Title::fixUrlQueryArgs Br

[MediaWiki-commits] [Gerrit] fix some spacing - change (mediawiki/core[master])

2013-02-01 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: fix some spacing .. fix some spacing Change-Id: I93f0a87ba7129bc336083e7289247d3150feb606 --- M includes/cache/CacheDependency.php M includes/cache/HTMLFileCach

[MediaWiki-commits] [Gerrit] Show no size links on image pages with errors - change (mediawiki/core[master])

2013-02-02 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: Show no size links on image pages with errors .. Show no size links on image pages with errors This avoids useless "Size of this preview: . Other resolutions:

[MediaWiki-commits] [Gerrit] Avoid undefined offset for $wgImageLimits - change (mediawiki/core[master])

2013-02-02 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: Avoid undefined offset for $wgImageLimits .. Avoid undefined offset for $wgImageLimits This can happen, when customized $wgImageLimits, but not the option in $w

[MediaWiki-commits] [Gerrit] ForeignAPIRepo: Show thumb error from foreign repo - change (mediawiki/core[master])

2013-02-02 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: ForeignAPIRepo: Show thumb error from foreign repo .. ForeignAPIRepo: Show thumb error from foreign repo When the api of the foreign repo gives a thumb error, t

[MediaWiki-commits] [Gerrit] fix some spacing - change (mediawiki/core[master])

2013-02-03 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: fix some spacing .. fix some spacing Change-Id: I670a7baf3ba1e70d18346bb00938d518ba2063bd --- M includes/actions/HistoryAction.php M includes/api/ApiBase.php M

[MediaWiki-commits] [Gerrit] fix some spacing - change (mediawiki/core[master])

2013-02-03 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: fix some spacing .. fix some spacing Change-Id: Ia9650be8b791fd2f0ccbc099436918a9d12d9968 --- M includes/installer/CliInstaller.php M includes/installer/Databas

[MediaWiki-commits] [Gerrit] fix some spacing - change (mediawiki/core[master])

2013-02-03 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: fix some spacing .. fix some spacing Change-Id: I88e73d47a552918880514d88a876296a6cb80d88 --- M includes/filebackend/FSFile.php M includes/filebackend/FileOp.ph

[MediaWiki-commits] [Gerrit] fix some spacing - change (mediawiki/core[master])

2013-02-03 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: fix some spacing .. fix some spacing Change-Id: I8f976013f33c5818e4402604fe8610aa3f43b0c6 --- M includes/ArrayUtils.php M includes/Article.php M includes/Block.

[MediaWiki-commits] [Gerrit] apitests leaks wgUser - change (mediawiki/core)

2013-02-08 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: apitests leaks wgUser .. apitests leaks wgUser 1) ParserTests::testParserTest with data set #468 ('pre-save transform: Signature expansion', '* ~~~ * ~~~ * ~~~

[MediaWiki-commits] [Gerrit] set wgLanguageCode in NewParserTests for all tests - change (mediawiki/core)

2013-02-08 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: set wgLanguageCode in NewParserTests for all tests .. set wgLanguageCode in NewParserTests for all tests Change one method default Change-Id: Id8d6817c616c2898

[MediaWiki-commits] [Gerrit] Title.php: replace some '*' with explicit fields in selects - change (mediawiki/core)

2013-02-09 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: Title.php: replace some '*' with explicit fields in selects .. Title.php: replace some '*' with explicit fields in selects It is good practice to select only fi

[MediaWiki-commits] [Gerrit] Add ArchivedFile::selectFields() and use it - change (mediawiki/core)

2013-02-09 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: Add ArchivedFile::selectFields() and use it .. Add ArchivedFile::selectFields() and use it Change-Id: Ia19d7bc1eb2bf77f1aca9d3891cebf57e45c030b --- M includes/f

[MediaWiki-commits] [Gerrit] Use OldLocalFile::selectFields() - change (mediawiki/core)

2013-02-09 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: Use OldLocalFile::selectFields() .. Use OldLocalFile::selectFields() It is good practice to specifiy the fields in a select Change-Id: I3faa568ed6311a585da9161

[MediaWiki-commits] [Gerrit] fix some spacing - change (mediawiki/core)

2013-02-09 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: fix some spacing .. fix some spacing Change-Id: I306091347ccaaf11dee0cdfda3019cb0c12be51b --- M includes/EditPage.php M includes/FeedUtils.php M includes/Html.p

[MediaWiki-commits] [Gerrit] Add some missing readonly checks: - change (mediawiki/core)

2013-02-09 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: Add some missing readonly checks: .. Add some missing readonly checks: * Do not allow working on Special:EditWatchlist * Do not reset all notification markers *

[MediaWiki-commits] [Gerrit] TRUE, FALSE and NULL must be lowercase - change (mediawiki/core)

2013-02-09 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: TRUE, FALSE and NULL must be lowercase .. TRUE, FALSE and NULL must be lowercase From phpcs Change-Id: I09672012be28ae3985702e314442db00d154de24 --- M includes

[MediaWiki-commits] [Gerrit] remove space before semicolon - change (mediawiki/core)

2013-02-09 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: remove space before semicolon .. remove space before semicolon Change-Id: Ic0d890f5e27cec017c7f7910a67d53b2edf82079 --- M includes/Article.php M includes/EditPa

[MediaWiki-commits] [Gerrit] remove not needed return and break - change (mediawiki/core)

2013-02-09 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: remove not needed return and break .. remove not needed return and break break after throw is not executed return; at end of function is not needed Change-Id:

[MediaWiki-commits] [Gerrit] ValidGlobalName: whitelist $messageMemc and $parserMemc - change (mediawiki...codesniffer)

2013-02-10 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: ValidGlobalName: whitelist $messageMemc and $parserMemc .. ValidGlobalName: whitelist $messageMemc and $parserMemc Both are in the docs under docs\globals.txt

[MediaWiki-commits] [Gerrit] fix hardcoded quote in BacklinkCache - change (mediawiki/core)

2013-02-10 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: fix hardcoded quote in BacklinkCache .. fix hardcoded quote in BacklinkCache Also make the IS NULL in upper case, looks better and sql reservered words often wr

[MediaWiki-commits] [Gerrit] SpecialPageFactory::resolveAlias returns null for invalid na... - change (mediawiki/core)

2013-02-10 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: SpecialPageFactory::resolveAlias returns null for invalid names .. SpecialPageFactory::resolveAlias returns null for invalid names Change-Id: I8def9dd88ebfbe9fc

[MediaWiki-commits] [Gerrit] Avoid $wgUser when getting tokens in api - change (mediawiki/core)

2013-02-11 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. Change subject: Avoid $wgUser when getting tokens in api .. Avoid $wgUser when getting tokens in api The api is using static user functions to obtain a token. This functions us

[MediaWiki-commits] [Gerrit] Pass user to File::recordUpload to avoid $wgUser - change (mediawiki/core)

2013-02-12 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/48658 Change subject: Pass user to File::recordUpload to avoid $wgUser .. Pass user to File::recordUpload to avoid $wgUser Moves the giving o

[MediaWiki-commits] [Gerrit] Add url param 'extended' to Special:Watchlist - change (mediawiki/core)

2013-02-12 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/48660 Change subject: Add url param 'extended' to Special:Watchlist .. Add url param 'extended' to Special:Watchlist This new url param allow

[MediaWiki-commits] [Gerrit] cleanup action=tokens - change (mediawiki/core)

2013-02-13 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/48871 Change subject: cleanup action=tokens .. cleanup action=tokens * profile of execute function is done by the caller * $type is always lo

[MediaWiki-commits] [Gerrit] [Regression] Do not trim spaces on edit preview or show chan... - change (mediawiki/core)

2013-02-17 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/49483 Change subject: [Regression] Do not trim spaces on edit preview or show changes .. [Regression] Do not trim spaces on edit preview or sh

[MediaWiki-commits] [Gerrit] deprecated two unused methods - change (mediawiki/core)

2013-02-23 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/50552 Change subject: deprecated two unused methods .. deprecated two unused methods Change-Id: I9e8847ec37f714037ea6867aa531c2b308a02f17 ---

[MediaWiki-commits] [Gerrit] Added missing wfDeprecated calls and @deprecated docs - change (mediawiki/core)

2013-02-23 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/50577 Change subject: Added missing wfDeprecated calls and @deprecated docs .. Added missing wfDeprecated calls and @deprecated docs Change-I

[MediaWiki-commits] [Gerrit] Remove AdminSettings.php hint from Installer message - change (mediawiki/core)

2014-02-28 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/116162 Change subject: Remove AdminSettings.php hint from Installer message .. Remove AdminSettings.php hint from Installer message Message 'c

[MediaWiki-commits] [Gerrit] Add missing DROP SEQUENCE to postgres' tables.sql - change (mediawiki/core)

2014-02-28 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/116166 Change subject: Add missing DROP SEQUENCE to postgres' tables.sql .. Add missing DROP SEQUENCE to postgres' tables.sql To drop all poss

[MediaWiki-commits] [Gerrit] Use a secondary db connection for Revsion::getParentLength i... - change (mediawiki/core)

2014-03-02 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/116286 Change subject: Use a secondary db connection for Revsion::getParentLength in api .. Use a secondary db connection for Revsion::getPare

[MediaWiki-commits] [Gerrit] Remove final period from api module descriptions - change (mediawiki/core)

2014-03-02 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/116290 Change subject: Remove final period from api module descriptions .. Remove final period from api module descriptions The most descripti

[MediaWiki-commits] [Gerrit] return array on prop=imageinfo for empty metadata - change (mediawiki/core)

2014-03-02 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/116291 Change subject: return array on prop=imageinfo for empty metadata .. return array on prop=imageinfo for empty metadata A array produces

[MediaWiki-commits] [Gerrit] Add @deprecated hint to $j - change (mediawiki/core)

2014-03-05 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/116977 Change subject: Add @deprecated hint to $j .. Add @deprecated hint to $j Change-Id: I0d9204ea354316ca11f568c81ad38f4089c91e80 --- M res

[MediaWiki-commits] [Gerrit] LocalSettings.php must be readable by the server - change (mediawiki/core)

2014-03-05 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/117022 Change subject: LocalSettings.php must be readable by the server .. LocalSettings.php must be readable by the server Bug: 61094 Change-

[MediaWiki-commits] [Gerrit] Remove wildcard actions from list=logevents&leaction= - change (mediawiki/core)

2014-03-05 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/117041 Change subject: Remove wildcard actions from list=logevents&leaction= .. Remove wildcard actions from list=logevents&leaction= The docu

[MediaWiki-commits] [Gerrit] Revert "Rollback: Make update of recentchanges table later" - change (mediawiki/core)

2014-03-06 Thread Umherirrender (Code Review)
Umherirrender has uploaded a new change for review. https://gerrit.wikimedia.org/r/117307 Change subject: Revert "Rollback: Make update of recentchanges table later" .. Revert "Rollback: Make update of recentchanges table late

<    1   2   3   4   5   6   7   8   9   10   >