Extension talk:SpecialDeleteOldRevisions2

From mediawiki.org
Latest comment: 10 years ago by 174.119.254.138 in topic Fixing security issues

Does not work with Postgres (Fixed)[edit]

If the extension is used with a Postgres database as backend, the following error occurs:

A database error has occurred
Query: SELECT rev_id FROM revision,page WHERE (rev_id NOT IN ( 10918, 10950, 10977, 6545,
 7219, 5893, 6399, 1655, 10411, 1931, 2704, 7876, 7882, 7878, 7880,
 6702, 6909, 6910, 6950, 6705 ) AND rev_timestamp <= '20100311999999')
 AND (rev_page = page_id AND page_title LIKE 'Z%')
Function: Database::select
Error: 1 ERROR: invalid input syntax for type timestamp with time zone: "20100311999999"

Obviously, '20100311999999' isn't a valid date. I assume MySQL is doing some magick behind the scenes so that it works nevertheless. Why is the '999999' appended anyway? Because just removing them seems to work fine.

Fixed in 1.4.1 Jehy 01:40, 23 March 2010 (UTC)Reply

I still get a similar error

 select rev_id FROM revision,page WHERE (rev_id NOT IN ( 2704, 1655, 10411, 1931, 
7876, 7878, 7880, 7882, 6702, 5893, 6705, 6545,  10950, 6399, 6909, 6910, 7219, 
10918, 10977, 6950 ) AND rev_timestamp <= '20100413235959') AND (rev_page = 
page_id AND page_title  LIKE 'Z%');
ERROR:  invalid input syntax for type timestamp with time zone: "20100413235959"

Assing a space between the day and the hour works fine: "20100413 235959"

20:36, 14 April 2010 (GMT+2)

Thank you for your reply. Could you tell me more - what type of field is used for storing this timestamp in postgres? In MySQL, it is binary field and obviously I can't add this space... Possibly, there is some workaround using mediawiki functions. Jehy 00:10, 16 April 2010 (UTC)Reply


80.65.225.88 09:18, 24 July 2010 (UTC)Reply

Putting a space doesn't work with me (I use mediawiki 1.15.4 with PostgreSQL 8.3.7, and version 1.4.1 of this extension). This is how rev_timestamp is defined:

Column           Type                         Not Null     Default
rev_timestamp    timestamp with time zone     NOT NULL

Sample value:

2010-07-23 19:27:15+00

hope this helps.

80.65.225.88 09:54, 24 July 2010 (UTC)Reply

Well. As a quick workarround, I replaced line 486:

 $maxDate = str_replace('-','',$this->mRequest->getText( 'wpMaxDate' )) . ' 235959';

with:

$maxDate = $this->mRequest->getText( 'wpMaxDate' )) . ' 23:59:59+00';

Next error is:

 Query: SELECT DISTINCTROW rev_text_id FROM revision
 Function:
 Error: 1 ERROR: syntax error at or near "rev_text_id"
 LINE 1: SELECT /* WikiSysop */ DISTINCTROW rev_text_id FROM revisio...
 ^
 
 Backtrace:
 
 #0 /filer/s/k/undisclosed/w/includes/db/Database.php(616): DatabasePostgres->reportQueryError
('ERROR: syntax ...', 1, 'SELECT DISTINCT...', '', false)
 #1 /filer/s/k/undisclosed/w/extensions/SpecialDeleteOldRevisions2/SpecialDeleteOldRevisions2.php(103):
 Database->query('SELECT DISTINCT...')
 #2 /filer/s/k/undisclosed/w/extensions/SpecialDeleteOldRevisions2/SpecialDeleteOldRevisions2.php(502):
 PurgeRedundantText(true)
 #3 /filer/s/k/undisclosed/w/extensions/SpecialDeleteOldRevisions2/SpecialDeleteOldRevisions2.php(530):
  DeleteOldRevisionsForm->execute()
 #4 /filer/s/k/undisclosed/w/includes/SpecialPage.php(559): DeleteOldRevisionsPage->execute(NULL)
 #5 /filer/s/k/undisclosed/w/includes/Wiki.php(229): SpecialPage::executePath(Object(Title))
 #6 /filer/s/k/undisclosed/w/includes/Wiki.php(59): MediaWiki->initializeSpecialCases(Object(Title), 
Object(OutputPage), Object(WebRequest))
 #7 /filer/s/k/undisclosed/w/index.php(116): MediaWiki->initialize(Object(Title), NULL, 
Object(OutputPage), Object(User), Object(WebRequest))
 #8 {main}

I guess this extension was written with MySQL in mind. Postgres doesn't support DISTINCTROW. By the way, I may be wrong, but it seems that DISTINCT is more appropriate in your script (I'm not MySQL nor PHP fluent, but it looks like you want to build a set as a filter, and DISTINCTROW may return useless duplicates).

The extension seems to work with Postgres when you change (line 103):

$res = $dbw->query( "SELECT DISTINCTROW rev_text_id FROM $tbl_rev" );

with:

$res = $dbw->query( "SELECT DISTINCT rev_text_id FROM $tbl_rev" );

and (line 111):

$res = $dbw->query( "SELECT DISTINCTROW ar_text_id FROM $tbl_arc" );

with

$res = $dbw->query( "SELECT DISTINCT ar_text_id FROM $tbl_arc" );

Table Optimization in 1.4.3 Breaks it Further (Fixed)[edit]

The added "Optimize Table" loop breaks it for any database other than MySQL, since MySQL specific commands are used. Other databased either have other methods to "optimize" tables or don't need that step at all.

Query failed: ERROR: unrecognized configuration parameter "tables" 
in /var/www/wiki/htdocs/mediawiki/includes/db/DatabasePostgres.php 
on line 580

I have commented out the optimization loop around line 369 to get it working.

Thanks, fixed this in 1.4.4 - now extension detects database type. Also, replaced "distinctrow" with "distinct" - it's really better, and it bugged you ealier :)Jehy 23:47, 20 September 2010 (UTC)Reply

Keep at least five or ten revisions, regardless of date[edit]

I would like to keep at least five or ten revisions per article. Regardless of how old the next-to-lat revision was. If Mr. Schmidt would add that function I would be grateful. Otherwise, I might take a look at how 'maxdate' is handled and add a 'maxcount' for this purpose. Thanks for making this extension. --John S. Peterson 16:25, 7 February 2010 (UTC)Reply

I agree, an option to leave the last 5 or so revisions would be really helpful and add great value to this extension. -Seb

Thanks[edit]

Great job. Thanks! It works fine on Mediawiki 1.14.0 with MySql 5.0.67-0ubuntu6 Enzo

Deleting only archived revisions[edit]

Is there any possibility of modifying this to allow deleting only deleted revisions, that is, the "archived" revisions that can be selected in the current version of the extension?

There are some times when it is helpful to keep a history, but when one would like to delete garbage revisions (e.g., revisions made during testing) and keep the size of the database down. Currently, first, I delete the page, then restore the revisions I want to keep, then I run the following MySQL queries, but they don't seem to get everything:

To list the deleted revisions:

SELECT `tblprefix_archive`.`ar_text_id`, `tblprefix_archive`.`ar_title`,`tblprefix_text`.`old_id`,`tblprefix_text`.`old_text`, `tblprefix_revision`.`rev_id` FROM tblprefix_archive, tblprefix_text, tblprefix_revision WHERE ((`tblprefix_text`.`old_id`=`tblprefix_archive`.`ar_text_id`) AND (`tblprefix_text`.`old_id`=`tblprefix_revision`.`rev_id`)) ORDER BY `tblprefix_text`.`old_id` DESC

To delete them:

DELETE FROM tblprefix_archive, tblprefix_revision, tblprefix_text USING tblprefix_archive, tblprefix_revision, tblprefix_text WHERE WHERE ((`tblprefix_archive`.`ar_text_id` >0) AND (`tblprefix_text`.`old_id`=`tblprefix_archive`.`ar_text_id`) AND (`tblprefix_text`.`old_id`=`tblprefix_revision`.`rev_id`))

Is this a reasonable approach that gets all the old text (and all references to it) out of the database?

If you don't have time to change the extension, I'd appreciate any tips about how I could modify the extension myself (or at least fix my SQL queries), but I'm more of a "front-end" user - I don't know much about PHP or MySQL.

Thanks! --Fungiblename 19:51, 2 February 2009 (UTC)Reply

There is a maintenance script available for this, see MediaWiki maintenance - removing deleted revisions. Patheticcockroach 15:05, 28 July 2009 (UTC)Reply
If you still need this, I can add an option. Jehy 03:02, 1 December 2009 (UTC)Reply

Strange error message[edit]

Hello, I get an error message, when trying to use this tool. the error message is partially in german language:

Es ist ein Datenbankfehler aufgetreten. Der Grund kann ein Timeout sein, der Ausfall eines Servers oder auch ein Programmierfehler. Die letzte Datenbankabfrage lautete:

(SQL-Abfrage versteckt)

aus der Funktion „“. MySQL meldete den Fehler „1064: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ' , , , , , , , , , , , , , , , , , , , , , , , , , , , 139, 140, 143, 144, 145, ' at line 1 (localhost)“.

Is it possible that this extension does not work for MySQL 5 and above? Or what is wrong?

Thanks for help! -- Roy, 12:05, 22 March 2009 (GMT)

Extension works fine for MySQL 5, we need to take a closer look at your problem. Jehy 03:02, 1 December 2009 (UTC)Reply

have the same problem. help? --Seb edit: it works when you UNCHECK "db integrity check" --seb edit 2: nope, doesn't, only works when you click "test" --seb


Try checking the database permissions of the user you are connecting with. I had the same error, tried everything, amongst the last things I tried and the only thing I hadn't tried before was giving the user full permissions on the database (it was quite restricted before). Now this script and the standard maintenance scripts that were having similar errors all work. - No idea why insufficient permissions would result in this error though. -- Stephen 01 March 2013.

Problems downloading extension (Fixed)[edit]

It appears as though Jehy's site is gone. Is there a backup site with the code?


Maybe its a temp downtime. --Choshi 15:52, 13 September 2009 (UTC)Reply
Yeah, sorry for this, all ok now. Thanks for providing backup link. Jehy 03:02, 1 December 2009 (UTC)Reply

pruning revisions using a "pruning date"[edit]

Hello,

I am very interested in a single parameter for your extension, a single date to prune (delete) the revision table before that date. I will pay a bounty (give you a donation) for such a modification (stable program version needed). --Wikinaut 22:47, 22 December 2009 (UTC)Reply

No problem, just contact me (see in my profile, how) Jehy 13:10, 27 December 2009 (UTC)Reply

Compatibility issue with SemanticForms 1.8.8 (Fixed)[edit]

See: this page for details. "HTMLForm.php" is deprecated and should be removed from the bundle. Alvinos 13:26, 8 March 2010 (UTC)Reply

Thank you, I changed class name so there should not be problems any more. Please update to 1.4.1 version. Jehy 01:25, 23 March 2010 (UTC)Reply

broken in latest svn of wiki (Fixed)[edit]

Fatal error: Call to undefined function wfElement() in /web/htdocs/www.www0.org/home/wiki/extensions/SpecialDeleteOldRevisions2/SpecialDeleteOldRevisions2.php on line 396 --212.54.219.23 11:17, 14 March 2010 (UTC)Reply

I went ahead and fixed it: version 1.4 fixed for SVN 63760). It was only an "xml::" format needed to be added on 2 functions. --Athinker 23:37, 14 March 2010 (UTC)Reply
Thank you, I just don't update my wiki :) I updated my archive, removed warning message and changed version to 1.4.1. Also I made several little bugfixes so you should update to 1.4.1 too :) Jehy 01:15, 23 March 2010 (UTC)Reply

Cannot redeclare deleteoldrevisions (Fixed)[edit]

C:\php5>php deleteOldRevisions.php

Fatal error: Cannot redeclare deleteoldrevisions() (previously declared in ...\extensions\SpecialDeleteOldRevisions2\SpecialDeleteOldRevisions2.php:147) in ...\maintenance\deleteOldRevisions.inc on line 68 --91.78.126.159 00:54, 5 April 2010 (UTC)Reply

don't work all Namespace in MW 1.15.1[edit]

Article name: %
Namespace: -100
Modus: Test only

Old revisions of 1086 pages need to be checked.
0 old revisions found.

but if choose specific, than it work

Article name: %
Namespace: 0
Modus: Test only

Old revisions of 275 pages need to be checked.
1 old revisions found.

--91.78.126.159 01:13, 5 April 2010 (UTC)Reply

[FIX] The latest change conflicts with HTMLForm at least here[edit]

Hi, I'm the guy that had made the change for SVN above recently.

In 1.4.1, at least in a local windows installation, HTMLForm.php isn't properly read so the wiki fails to load [with PHP Fatal error: Class 'DeleteOldRevisions_HTMLForm' not found].

However, it is fixed if also HTMLForm.php is renamed to DeleteOldRevisions_HTMLForm.php and that is also reflected in the appropriate 'require_once'. --Athinker 21:50, 6 April 2010 (UTC)Reply

PS. What is also peculiar is that it worked if there was no rename and Class was returned to previous; Apparently there was already an HTMLForm on the system used correctly.

That's because you didn't overwrite HTMLForm.php with newer version at first time. In 1.4.1, file name for inclusion is "HTMLForm.php" and the class name is "DeleteOldRevisions_HTMLForm" - everything is fine with it. Jehy 16:20, 7 April 2010 (UTC)Reply
That can clearly not be the case since I renamed it and included it and it worked. If I renamed it and included the same old file it wouldn't work since it would call a wrong function.

Strict Standards error (Fixed)[edit]

in

MediaWiki  	1.15.2
PHP 	5.3.2 (apache2handler)
MySQL 	5.1.45

running on

XAMPP for Linux 1.7.4-beta2

i kept getting

Strict Standards: Declaration of DeleteOldRevisionsPage::execute() should be compatible with that of SpecialPage::execute()in /opt/lampp/htdocs/mediawiki/unix/extensions/SpecialDeleteOldRevisions2/SpecialDeleteOldRevisions2.php on line 512

errors, until Bryan from #mediawiki@freenode.net suggested i change line 512 in DeleteOldRevisions2.php

from

    function execute() {

to

    function execute( $par = null ) {
Thank you, added your fix in version 1.4.3 Jehy 09:49, 14 September 2010 (UTC)Reply
Had to add this into my copy (downloaded today) as it appears to have gone missing — GrahamSmith 11:09, 20 April 2011 (UTC)Reply

Broken in 1.16 (?) (Fixed)[edit]

Loading this in 1.16 results in a crashed wiki.

"Fatal error: Class 'DeleteOldRevisions_HTMLForm' not found in ........\extensions\SpecialDeleteOldRevisions2\SpecialDeleteOldRevisions2.php on line 371"

Have not modified the code or the LocalSettings line in any way. Bizarre, because my eyes can find the class definition just fine...

Please advise!

-- 19:58, 14 June 2010 (UTC) DoubleSuitedChris

Getting the same problem

Solved in version 1.4.2, by User:Ff9will

Broken in 1.16 bis (?)[edit]

For me it doesn't even appear in special pages, it worked fine in 1.15.5. I am bureaucrat.

Not deleting old versions of in the File namespace[edit]

I installed this extension with a view to deleting old versions of files (Images). I ran the extension which worked perfectly in removing the revision history from pages, but looking at File pages, the old versions are still there. This means when I clean out the images/archive directory I will have pages linking to non-existant files.

Has anyone else had this issue? --67.247.57.151 14:35, 25 July 2010 (UTC)Reply


Extension name in Special Pages List[edit]

In 1.16 the name of the extension is listed as "<deleteoldrevisions>" and on the special page of the extension the heading is "<deleteoldrevisions>" as well! Page title is the same thing. Any suggestions? ---- 2nd August 2010

[Fix 1 of 3] $wgOut->setPageTitle( 'Special Page:DeleteOldRevisions' );

This extension uses $wgMessageCache->addMessage(). That method is deprecated in MediaWiki 1.16. It should still work though, as far as I can tell. Reach Out to the Truth 15:26, 2 August 2010 (UTC)Reply
Fixed in version 1.4.3 Jehy 09:45, 14 September 2010 (UTC)Reply

Still doesn't work for me (and my 1.16.), even with Version 1.4.4 (which claims to be 1.4.3). Page title, Special Pages List and submit buttons all show placeholders instead of Text. I still see $wgMessageCache->addMessage in the code. So that works for others? Michael

Does not work for me either. I had to add messages in "MessagesFr.php" files, as this is described for language tips, Etienne.

not working[edit]

results in a blank page with mediawiki 1.15.x, any ideas?

Hello. Fixed in 1.4.3? Jehy 09:45, 14 September 2010 (UTC)Reply

Doesn't work with high number of artices[edit]

Article name: % Namespace: -100 Modus: Test only

Old revisions of 1216 pages need to be checked. 0 old revisions found.

No changes have been made.


Article name: % Namespace: 0 Modus: Test only

Old revisions of 402 pages need to be checked. 7 old revisions found.

No changes have been made.


Probably, the problem with the query, which has 1216 values in IN(), but in MySQL docs: The number of values in the IN list is only limited by the max_allowed_packet value. http://dev.mysql.com/doc/refman/5.0/en/comparison-operators.html#function_in

I've changed max_allowed_packet from 1MB to 50MB and to 512MB, but it didn't solve the problem.

Any ideas how to solve this? Thanks in advance. --217.12.216.18 07:54, 13 August 2010 (UTC)Reply

Hello. Please, locate the SQL query which is too long, I'll try to do smth with it.Jehy 09:42, 14 September 2010 (UTC)Reply

[FIX] Encoding and Inheritance problem (2010/08/21) - 1.4.2[edit]

That bug:
"Fatal error: Class 'DeleteOldRevisions_HTMLForm' not found in ROOT/extensions/SpecialDeleteOldRevisions2/SpecialDeleteOldRevisions2.php on line 371"
was a encoding problem. Other minimal bugs are fixed, and now are working with 1.6+
The fix is packed as 1.4.2.

Preserving page history but not revisions?[edit]

How about only deleting the data in the field 'old_text' (table `mw_text`) for old revisions, instead of deleting that whole row? This will preserve the page history and still reduce the DB size. The links wont work any more or perhaps will only show changes from 0 Bytes to the next but thats ok. I really dont like loosing the page histories. --Choshi 02:15, 12 September 2010 (UTC)Reply

Hi. I don't see any meaning to this:) Why do you want to preserve editing metadata but not revicions itself? What use has metadata without the article text? Jehy 09:41, 14 September 2010 (UTC)Reply
With what it is right now, everything is deleted. We dont know when the page was created and who edited and when. The only thing we have left is the last editor. All other information is lost. Often we want to see who has edited the page and when, when it was created. If the diff link doesnt work, thats still ok because we would have at least some information in the history. So if the field 'old_text' (table `mw_text`) was set to null or zero, that would do the job. I believe in that way, you would also not have to change/delete any other tables since the main data that takes up space has been deleted and we keep everything else.
What do you think? I understand this means changing a code a lot and maybe you dont agree this is a good way. I'll try to play with your code and see if I can come up with something. The way I would do it is: 1) set old text = '0/null' 2) add "diff not available (article size in KB)" in the end or start of the edit summary of that certain revision to let people know they cant see the changes, but everything else is still there. --Choshi 15:20, 15 September 2010 (UTC)Reply


Blank page on MW 1.16.0[edit]

10-20-2010 5:07PM EST
It works on WINXP XAMPP v1.70 on locahost but not on a hosted environment. I am using a subdomain, i.e. wiki.example.com. Everything else works fine but cannot get this extension to work.
My LocalSettings.php

 $wgScriptPath  = "";
 $wgArticlePath = "$wgScriptPath/$1";

There is a problem with the folder name of the extension. Correct the folder name (with the right LetterCase) will fix the problem. WINXP is case insensitive, so it worked on it.--Superxain 18:30, 31 March 2011 (UTC)Reply

where I was wrong[edit]

Fatal error: require_once() [function.require]: Failed opening required '/home/zave/public_html/kupus/extensions/SpecialDeleteOldRevisions2/SpecialDeleteOldRevisions2.php' (include_path='/home/zave/public_html/kupus:/home/zave/public_html/kupus/includes:/home/zave/public_html/kupus/languages:.:/usr/local/php52/pear') in /home/zave/public_html/kupus/LocalSettings.php on line 133

wiki version 1.16 SpecialDeleteOldRevisions2 1.44 Bonzo 13:48, 25 December 2010 (UTC)Reply

There is a problem with the folder name of the extension. Correct the folder name (with the right LetterCase) will fix the problem. --Superxain 18:31, 31 March 2011 (UTC)Reply

Response to error reports above[edit]

If anyone encountered an error during installation (a blank screen, for instance), then there may be a simple mismatch between the names of the files you downloaded and the lines to be inserted in localsettings (which is case sensitive). Rename the folder ("specialdeleteoldrevisions2") to "SpecialDeleteOldRevisions2" and if you're still experiencing trouble, rename the line beginning "require_once" as follows:

require_once( $IP.'/extensions/SpecialDeleteOldRevisions2/SpecialDeleteOldRevisions2.php');
to
require_once( "$IP/extensions/SpecialDeleteOldRevisions2/SpecialDeleteOldRevisions2.php");

Cavila 20:15, 25 January 2011 (UTC)Reply

Two questions[edit]

  1. When an article title is specified for "View content page", the query finds no old revisions and nothing is deleted. Is that normal?
  2. If an article was last revised before the cut-off date, will it still preserve the latest version (I guess so but I'm asking just make sure).

Regards, Cavila 20:15, 25 January 2011 (UTC)Reply

Empty table handling[edit]

If you misspell the name of the article to one that does not exist, you get an "undefined variable" error message ('$cur').

Likewise, if it's not in the archive, for '$arc'. (MW 1.16.2, MySQL 5.1.30)

Mickeyf 18:59, 14 February 2011 (UTC)Reply

Just try this right before the line where the error occured in SpecialDeleteOldRevisions2.php:
if (!isset($cur)) $cur = NULL;
This could be repeated for the variables $oldText, $old, $arc

MrBr4u53 07:40, 16 March 2012 (UTC)Reply

Scripting the php[edit]

Can this be modified in order to use it as a script from command line?

Thank you, Cristi 79.112.67.68 22:13, 28 February 2011 (UTC)Reply

Still getting a blank page[edit]

I'm still having trouble getting this to work on Ubuntu 10.04 LTS, using MySQL. I have made the suggested changes to LocalSettings.php:

$wgGroupPermissions['sysop']['DeleteOldRevisions'] = true;
require_once( "$IP/extensions/SpecialDeleteOldRevisions2/SpecialDeleteOldRevisions2.php");

The folder and file names are correct, with correct capitalization. I still only get a blank page. I have one other extension, UserMerge, which shows up fine. Any suggestions? WDavis1911 18:04, 1 April 2011 (UTC)Reply


Actually when I downloaded this, the folder was not capitalized. Double check that the folder you uploaded actually has the same capitalization as this tag. I think the article should be changed or the download should be checked.—The preceding unsigned comment was added by 173.35.238.108 (talk • contribs) date. Please sign your posts with ~~~~!

Thanks for responding. I have checked the capitalization, including doing a direct copy paste, many times. It is not the capitalization. I am now using the following versions of the Wiki stack:

  • MediaWiki: 1.17.0
  • PHP: 5.3.2-1ubuntu4.9 (apache2handler)
  • MySQL: 5.1.41-3ubuntu12.10

I have also added the Cite, and ParserFunctions (Version 1.3.0) extension. Hope this helps! WDavis1911 02:48, 5 July 2011 (UTC)Reply

Alright, I found the error, with the help of some php debugging. It was merely the directory permissions of SpecialDeleteOldRevisions2. They were 700, but needed to be 775. WDavis1911 03:13, 5 July 2011 (UTC)Reply

Thank-you[edit]

Script was just what i needed. After some serious spammers, i had a 44MB compressed DB. After deleting the revisions, i was down to 22.3.

Great job guys.

from the main page[edit]

Bug: BLANK PAGE AFTER SETUP ON DISPLAY!

moved here. Okip 01:25, 26 April 2011 (UTC)Reply

Check the capitalization of the folder you uploaded and make sure it matches the filename you added in LocalSettings.php

fatal error[edit]

message after installation...

Fatal error: Call to undefined method SpecialPage::specialpage() in /home/www/.../html/wikilis/extensions/SpecialDeleteOldRevisions2/SpecialDeleteOldRevisions2.php on line 535

the line 535 is...

SpecialPage::SpecialPage('DeleteOldRevisions', 'DeleteOldRevisions'); //$wgGroupPermissions['sysop']['DeleteOldRevisions'] = true;

i use mediawiki 1.17.0 on webspace of an provider under PHP Version 5.2.4-2ubuntu5.17
mediawiki local under WAMPP installed there is only an warning

Strict Standards:Declaration of DeleteOldRevisionsPage::execute() should be compatible with that os SpedialPage::execaute() in C:\xampp\htdocs\localwiki\extensions\specialdeleteoldrevisions2\SpecialDeleteOldRevisions2.php on line 532

anyway... the extension do his work on the local installation under Windows XP
thanks for help.. greatings... sorry --62.226.118.160 08:33, 4 August 2011 (UTC)Reply

Delete the editing history as well ?[edit]

The extension only remove the history revision of the page, but it not deleting the associated editing history lines. --Rabin .Y 19:36, 29 October 2011 (UTC)Reply

Doesn't work with 1.18[edit]

I have this argument on the top of the pages : Warning: call_user_func(wfExtensionSpecialDeleteOldRevisions) [function.call-user-func]: First argument is expected to be a valid callback in /home/users/m/mysite/www/includes/Setup.php on line 496 and the extension don't appear in the special pages. --Henrique Diaz 12:32, 22 December 2011 (UTC)Reply


Doesn't work with 1.18 (other reason)[edit]

I installed the extension, checked the capitalization of the folder like it was mentioned in talks above and the rights to access the script are set. But i still get an error message:

Fatal error: Call to a member function addMessage() on a non-object in C:\SEU\BitNami MediaWiki Stack\apps\mediawiki\htdocs\extensions\specialdeleteoldrevisions2\SpecialDeleteOldRevisions2.php on line 560
My installation is under Vista / Product Version = MediaWiki 1.18.0 / PHP 5.3.8 (apache2handler) / MySQL 5.5.16

Upgrade from 1.17 to 1.18 failed with extension enabled[edit]

I got the folowwing message:

#php update.php --conf /etc/mediawiki/LocalSettings.php
PHP Fatal error:  Call to a member function addMessage() on a non-object 
in /var/lib/mediawiki-1.18.0/extensions/specialdeleteoldrevisions2/SpecialDeleteOldRevisions2.php on line 560

I disabled it in rder to perform the Mediawiki upgrade, and leave it as disabled.

error on MediaWiki 1.18 on Windows[edit]

  • I also got only blank pages when I required the script. For me the error was not due to letter mimatch, but the "$wgMessageCache->addMessage" did not work. I commented all those lines, and the extension works (though with cryptic button labels) -- Forentw 20:15, 8 January 2012 (UTC)Reply

Same experience here. I'm running on Ubuntu. Commenting the "$wgMessageCache->addMessage" allows me to load correctly. A few notes:

  • I can get to the DeleteOldRevisions page
  • The buttons labels are garbled. Specifically, the button for "delete old revisions" button shows the label "&lt:deleteoldrevisions-button&gt:" (exchange the colons for semi-colons. I was having formatting issues). I would assume that &lt and &gt are some representation of "<" and ">".
  • The "test only" command seems to work properly.
  • The "delete old revisions" button hangs on a blank screen. Reloading shows that it has not deleted the old revisions.

Hope this helps... WDavis1911 05:59, 13 January 2012 (UTC)Reply

I have a fix in store for garbled labels, including the addMessage fix. It does work for me with no blank screen when I click "delete old revisions" button. Maybe it's timing out? Is your wiki really big with tons of old revisions? Try choosing a date last year (or far in the past) so it doesn't have to delete as much data and see if it fixes things. --Frugen (talk) 19:19, 10 September 2012 (UTC)Reply

SQL syntax error on MediaWiki 1.15[edit]

SELECT old_id FROM `text` WHERE old_id NOT IN ( 16911, 16912, 17340, 17518, 17519, ...,
20148, , 17820, 18752, 1325, 1326, 17758, 130, 17538, 17619, 16853, 16861, 16862, 16863, 19922 ) 
...
1064: You have an error in your SQL syntax;
check the manual that corresponds to your MySQL server version for the right syntax to use near
' 17820, 18752, 1325, 1326, 17758, 130, 17538, 17619, 16853, 16861, 16862, 16863,' at line 1 (localhost)“.

See the empty value between

20148, , 17820

Any idea, what is wrong ? This Wiki has 20159 revisions in the database for a few content pages. I think, something went wrong with the database in the past, but I do not know what. I wanted to use SpecialDeleteOldRevision2 to get rid of all old revisions.

Content pages	44
(All pages in the wiki, including talk pages, redirects, etc.)	244
Uploaded files	88
Page edits since the wiki was set up 20,159 <<<< this is very strange and certainly wrong
Average edits per page	82.6

--Wikinaut 23:29, 12 January 2012 (UTC)Reply

Tried to update this for 1.9.2[edit]

I tried to update this for 1.19.2 download. I did a few quick checks and it seemed to work fine. Ignore any errors at the top talking about undefined variables arc or old, etc.

This needs a fix for other languages, I forgot to do that and was too tired to do other stuff. I might check on it again if it doesn't work fine for me. Remember to make a backup of your database before you use it. It may be some time before I'm able to run this on an actual installation. Hopefully it will run at that time. If it doesn't, I might try to fix it and will report back here then. I changed the filenames for consistency/standardization etc (in accordance with Manual:Special pages). See the installation instructions in the section for 1.19.2. Good luck. --Choshi (talk) 06:59, 16 September 2012 (UTC)Reply

Fixed for 1.19.2[edit]

Hi. I came back to this extension, and fixed it for 1.19.

Last version seems stable - but still, PLEASE BACKUP BEFORE USING IT!

Also, now it's published on github.

P.S. Choshi, sorry, I may have lost some of your edits - please, collaborate with me on github.

Jehy (talk) 20:44, 17 September 2012 (UTC)Reply

Jehy - it seems you took up this project while I was working on it as well. I have made a number of changes to bring this code into newer mediawiki standards, including language support, reduced use of globals, etc. I'll Show you what I've got on github. --Frug (talk) 21:32, 19 September 2012 (UTC)Reply
That's cool, it will become even better :) Jehy (talk) 04:35, 20 September 2012 (UTC)Reply
I hope so. I've branched you on github so check that and let me know. --Frug (talk) 19:04, 20 September 2012 (UTC)Reply
Revieved and pushed your commit to original repo. Cool, thanks! Jehy (talk) 10:32, 25 September 2012 (UTC)Reply
Great work, thanks guys. I dont know how to use Git but I will see the next time I have to use this extension. --Choshi (talk) 03:14, 9 May 2013 (UTC)Reply

Small bug[edit]

To get the right description in 'Special:Version', the correct entry in $wgExtensionCredits for description should be 'descriptionmsg' and not just 'description'.
Thanks for your contributions!

Another small bug[edit]

If no pages are found reports the error:

Notice: Undefined variable: cur in …\DeleteOldRevisionsPage.php on line 235

In the function DeleteOldRevisions(...)
  The variable $cur must be properly initialized before the while, like

//Leo
$cur= null;
//\Leo
while( $row = $dbw->fetchObject( $res ) ) {
...

Regards!
  Leo

Thank you, both bugs fixed! Jehy (talk) 07:46, 19 November 2012 (UTC)Reply

Mediawiki 1.17.2 throws an exception (FIX)[edit]

Hi, Thanks for great extension. I cloned latest revision of extension and I got exception with 1.17.2 mediawiki.

[Sun Mar 17 14:34:55 2013] [error] [client 1.2.3.4] FastCGI: server "/var/www/php5.external" stderr: PHP message: PHP Fatal error:
Call to a member function isAllowed() on a non-object in /var/www/mediawiki-1.17.2/includes/SpecialPage.php on line 812, referer: https://server/index.php/Special:SpecialPages I fixed it by:

diff --git a/mediawiki/includes/SpecialPage.php b/mediawiki/includes/SpecialPage.php
index 12ad517..28a1b29 100755
--- a/mediawiki/includes/SpecialPage.php
+++ b/mediawiki/includes/SpecialPage.php
@@ -809,7 +809,8 @@ class SpecialPage {
         * @return Boolean: does the user have permission to view the page?
         */
        public function userCanExecute( $user ) {
-               return $user->isAllowed( $this->mRestriction );
+                global $wgUser;
+               return $wgUser->isAllowed( $this->mRestriction );
        }

        /**

And

diff --git a/mediawiki/extensions/DeleteOldRevisions2/DeleteOldRevisionsPage.php b/mediawiki/extensions/DeleteOldRevisions2/DeleteOldRevisionsPage.php
index c05ac3a..49718eb 100644
--- a/mediawiki/extensions/DeleteOldRevisions2/DeleteOldRevisionsPage.php
+++ b/mediawiki/extensions/DeleteOldRevisions2/DeleteOldRevisionsPage.php
@@ -21,9 +21,10 @@ class DeleteOldRevisionsPage extends SpecialPage {
                }

         $this->setHeaders();
-
-               if ( $this->getUser()->isBlocked() ) {
-                       $block = $this->getUser()->getBlock();
+               // KSt XXXX
+                global $wgUser;
+               if (  $wgUser->isBlocked() ) {
+                       $block = $wgUser->isBlocked()->getBlock();
                        throw new UserBlockedError( $block );
                }
                $this->checkReadOnly();

Thanks! -KSt

Fixing security issues[edit]

According to the deletion message see this, the direct SQL commands should be re-done using the Mediawiki method. I tried looking into it but I dont know how to fix it completely. Here's a temp download link in case anyone wants to look at it and fix it. The version I have is 1.6.1 which I downloaded in May 2013. It should be the latest. Personally I'm in no hurry to see a fix since I wont use it in the immediate future (butwill eventually). Others may need it so I'm posting the link in case anyone wants to try to fix it.

These pages have additional info on how to fix this: SQL injection, and Manual:Database access. --Choshi (talk) 12:12, 28 July 2013 (UTC)Reply

it's actually not that hard to fix, but I guess nobody has the time or desire to bother (myself included). If I do find a solution to deleting old revisions (there is a maintenance script that does the same thing, but none of the plugins using it work either!) I'll try to remember to post it here.

--174.119.254.138 00:03, 22 October 2013 (UTC)Reply