Page MenuHomePhabricator
Create Task
Maniphest T189985

image_comment_temp entries aren't being moved when a file is renamed
Closed, ResolvedPublic
Actions

Description

When a file is renamed, img_name is being changed but the corresponding entry in image_comment_temp is not.

Details

ProjectBranchLines +/-Subject
mediawiki/corewmf/1.31.0-wmf.25+11 -0Move image_comment_temp entries when the file is moved
mediawiki/coremaster+11 -0Move image_comment_temp entries when the file is moved
Customize query in gerrit

Related Objects

Event Timeline

Anomie created this task.Mar 18 2018, 2:44 PM
Restricted Application added projects: Commons, Multimedia. · View Herald TranscriptMar 18 2018, 2:44 PM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
Anomie triaged this task as Unbreak Now! priority.Mar 18 2018, 3:39 PM
Restricted Application added subscribers: Liuxinyu970226, TerraCodes. · View Herald TranscriptMar 18 2018, 3:39 PM
gerritbot added a subscriber: gerritbot.Mar 18 2018, 3:39 PM

Change 420249 had a related patch set uploaded (by Anomie; owner: Anomie):
[mediawiki/core@master] Move image_comment_temp entries when the file is moved

https://gerrit.wikimedia.org/r/420249

gerritbot added a project: Patch-For-Review.Mar 18 2018, 3:39 PM
TTO added a subscriber: TTO.Mar 19 2018, 1:35 AM

Scary that we're still using image names as primary keys...

gerritbot added a comment.Mar 19 2018, 1:44 AM

Change 420249 merged by jenkins-bot:
[mediawiki/core@master] Move image_comment_temp entries when the file is moved

https://gerrit.wikimedia.org/r/420249

ReleaseTaggerBot added a project: MW-1.31-release-notes (WMF-deploy-2018-03-20 (1.31.0-wmf.26)).Mar 19 2018, 2:00 AM
Rehman added a subscriber: Rehman.Mar 19 2018, 4:08 AM
Anomie added a comment.Mar 19 2018, 11:40 AM
In T189985#4059794, @TTO wrote:

Scary that we're still using image names as primary keys...

I know. T28741: Migrate file tables to a modern layout (image/oldimage; file/file_revision; add primary keys) seems to be the task for that.

gerritbot added a comment.Mar 19 2018, 1:12 PM

Change 420322 had a related patch set uploaded (by Anomie; owner: Anomie):
[mediawiki/core@wmf/1.31.0-wmf.25] Move image_comment_temp entries when the file is moved

https://gerrit.wikimedia.org/r/420322

gerritbot added a comment.Mar 19 2018, 1:34 PM

Change 420322 merged by jenkins-bot:
[mediawiki/core@wmf/1.31.0-wmf.25] Move image_comment_temp entries when the file is moved

https://gerrit.wikimedia.org/r/420322

Stashbot added a subscriber: Stashbot.Mar 19 2018, 1:44 PM

Mentioned in SAL (#wikimedia-operations) [2018-03-19T13:44:43Z] <anomie@tin> Synchronized php-1.31.0-wmf.25/includes/filerepo/file/LocalFile.php: Applying fix for T189985 (duration: 00m 58s)

Stashbot added a comment.Mar 19 2018, 1:48 PM

Mentioned in SAL (#wikimedia-operations) [2018-03-19T13:48:07Z] <anomie> Cleaning up orphaned image_comment_temp rows on all wikis for T189985

Anomie closed this task as Resolved.Mar 19 2018, 1:53 PM
ReleaseTaggerBot edited projects, added MW-1.31-release-notes (WMF-deploy-2018-03-13 (1.31.0-wmf.25)); removed MW-1.31-release-notes (WMF-deploy-2018-03-20 (1.31.0-wmf.26)).Mar 19 2018, 2:00 PM
Liuxinyu970226 removed a subscriber: Liuxinyu970226.Mar 20 2018, 3:34 AM
Ladsgroup removed a project: Patch-For-Review.May 21 2019, 9:33 PM
Aklapper removed a subscriber: Anomie.Oct 16 2020, 5:38 PM
Content licensed under Creative Commons Attribution-ShareAlike 3.0 (CC-BY-SA) unless otherwise noted; code licensed under GNU General Public License (GPL) or other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL