"Tim Starling" changed the status of MediaWiki.r113312 to "ok"
URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/113312

Old status:  new
New status: ok

Commit summary for MediaWiki.r113312:

[FileRepo]
* Made upload() explicitly check $status->successCount rather than isOk() as 
FileRepo::publish() gives fatal statuses where it used to only give warnings. 
This way, failed uploads still displace the current image DB row to oldimage 
rather than do nothing and have the same image row point to a new file (which 
can cause mismatched metadata).
* Disabled exception about invalid oi_archive_name. This makes things more 
broken in that when people get this an error on upload (which happens when the 
FS has no current file but the DB does), the new file is added as the current 
version in the FS but the DB is unchanged. Thus, the metadata can be mismatched.

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to