Last modified: 2012-05-28 17:56:47 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T30677, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 28677 - Fail more graciously when an expected existing revision, doesn't exist
Fail more graciously when an expected existing revision, doesn't exist
Status: NEW
Product: MediaWiki extensions
Classification: Unclassified
CodeReview (Other open bugs)
unspecified
All All
: Low minor (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-04-23 19:34 UTC by Sam Reed (reedy)
Modified: 2012-05-28 17:56 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Sam Reed (reedy) 2011-04-23 19:34:05 UTC
http://www.mediawiki.org/wiki/Special:Code/MediaWiki/83679

Failed to load expected revision data
Backtrace:
#0 /usr/local/apache/common-local/php-1.17/extensions/CodeReview/ui/CodeRevisionView.php(24): CodeRepository->getRevision(83679)
#1 /usr/local/apache/common-local/php-1.17/extensions/CodeReview/ui/SpecialCode.php(90): CodeRevisionView->__construct('MediaWiki', '83679')
#2 /usr/local/apache/common-local/php-1.17/extensions/CodeReview/ui/SpecialCode.php(24): SpecialCode->getViewFrom('MediaWiki/83679')
#3 /usr/local/apache/common-local/php-1.17/includes/SpecialPage.php(578): SpecialCode->execute('MediaWiki/83679')
#4 /usr/local/apache/common-local/php-1.17/includes/Wiki.php(251): SpecialPage::executePath(Object(Title))
#5 /usr/local/apache/common-local/php-1.17/includes/Wiki.php(63): MediaWiki->handleSpecialCases(Object(Title), Object(OutputPage), Object(WebRequest))
#6 /usr/local/apache/common-local/php-1.17/index.php(114): MediaWiki->performRequestForTitle(Object(Title), NULL, Object(OutputPage), Object(User), Object(WebRequest))
#7 /usr/local/apache/common-local/live-1.5/index.php(3): require('/usr/local/apac...')
#8 {main}


It's essentially been oversighted, that and r82952, so should fallback a bit more graciously...
Comment 1 Brion Vibber 2011-04-25 20:21:51 UTC
Missing revision data would normally be considered a serious bug as it's inconsistent with the repository state and subversion's versioning model.

Requests for revisions that haven't yet been reached are handled gracefully; for those that should be in the database we throw an exception as it indicates database corruption.

Are those revisions actually removed from SVN as well, and does SVN behave cleanly that way? If not, is there a possibility that some batch process will just fill them back in some day when nobody's thinking about it?
Comment 2 Roan Kattouw 2011-04-25 20:28:01 UTC
(In reply to comment #1)
> Are those revisions actually removed from SVN as well,
Yes.

> and does SVN behave
> cleanly that way? 
Sort of. The import script used to barf on the svn log output for removed revs, I fixed that a few days ago.

> If not, is there a possibility that some batch process will
> just fill them back in some day when nobody's thinking about it?
No way. SVN simply will not give you the data.
Comment 6 Mark A. Hershberger 2011-08-12 18:49:53 UTC
still happening :(
Comment 7 Mark A. Hershberger 2012-05-28 17:56:47 UTC
Lowering priority on high priority bugs that have a low severity

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links