[Bug 17237] Better integration of patrol feature (rc_patrolled, rc_bot in revision: rv_bot, rv_patrolled)
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 Daniel Kinzler changed: What|Removed |Added See Also||https://bugzilla.wikimedia. ||org/show_bug.cgi?id=47415 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 17237] Better integration of patrol feature (rc_patrolled, rc_bot in revision: rv_bot, rv_patrolled)
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 Daniel Kinzler changed: What|Removed |Added See Also||https://bugzilla.wikimedia. ||org/show_bug.cgi?id=25799 -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 17237] Better integration of patrol feature (rc_patrolled, rc_bot in revision: rv_bot, rv_patrolled)
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 Krinkle changed: What|Removed |Added CC||daniel.kinz...@wikimedia.de ||, tstarl...@wikimedia.org --- Comment #6 from Krinkle --- Progress from Amsterdam Hackathon 2013: * Move rc_patrolled and rc_bot from recentchanges to revision. * Make rev_patrolled 0/1/2 instead of 0/1. The value 2 would represent autopatrolled. * We no longer need logging for autopatrol because author/timestamp is the same as the author. this solves two bugs: Bug 25799 (Allow Special:Log to show separate autopatrol from human patrol) BUg 47415 (Logging of autopatrol does not scale for Wikidata) +1 from Daniel Kinzler, Tim Starling and Timo Tijhof. -- You are receiving this mail because: You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 17237] Better integration of patrol feature (rc_patrolled, rc_bot in revision: rv_bot, rv_patrolled)
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 Krinkle changed: What|Removed |Added Summary|Better integration of |Better integration of |patrol feature |patrol feature |(rc_patrolled, rc_bot in|(rc_patrolled, rc_bot in |revisions rv_bot, |revision: rv_bot, |rv_patrolled) |rv_patrolled) -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 17237] Better integration of patrol feature (rc_patrolled, rc_bot in revisions rv_bot, rv_patrolled)
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 Krinkle changed: What|Removed |Added Summary|Better integration of |Better integration of |patrol feature |patrol feature ||(rc_patrolled, rc_bot in ||revisions rv_bot, ||rv_patrolled) -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 17237] Better integration of patrol feature
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 Krinkle changed: What|Removed |Added Component|General/Unknown |Patrolling -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 17237] Better integration of patrol feature
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 p858snake changed: What|Removed |Added Blocks||16012 Depends on|16012 | -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 17237] Better integration of patrol feature
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 Krinkle changed: What|Removed |Added CC||krinklem...@gmail.com --- Comment #5 from Krinkle 2010-11-16 21:14:22 UTC --- One thing to not forget is wgRCMaxAge, the patrols expire after wgRCMaxAge. If moving this into recentchanges: * the rev_patrolled needs to be NULL'ed if 0 after wgRCMaxAge (or introduce a new variable wgMaxPatrolAge which by default is equal to wgRCMaxAge) * when patrolling an edit the markpatrolled action need to update both revision and recentchanges -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 17237] Better integration of patrol feature
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 ^demon changed: What|Removed |Added CC||innocentkil...@gmail.com Depends on||16012 -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 17237] Better integration of patrol feature
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 Roan Kattouw changed: What|Removed |Added Blocks||12394 -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 17237] Better integration of patrol feature
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 Roan Kattouw changed: What|Removed |Added Keywords||need-review, patch, schema- ||change -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 17237] Better integration of patrol feature
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 --- Comment #4 from Roan Kattouw 2009-02-02 15:26:32 UTC --- Created an attachment (id=5768) --> (https://bugzilla.wikimedia.org/attachment.cgi?id=5768) Proposed patch Attached patch that: * adds the rev_bot and rev_patrolled fields to the revision table * adds the populateRevBotPatrolled.php script that populates them (automatically run from update.php) * adds support for the fields in the Revision class ** deprecates isUnpatrolled() which seems to be unused in core * sets rev_bot and rev_patrolled appropriately when creating new revision entries This patch doesn't synchronize rev_bot and rc_bot for new moves, deletions and protections, because the interface for moving/deleting/protecting with the bot flag sucks (I intend to fix that) and doesn't sync them for old ones either because the recentchanges entry and the associated null revision aren't linked in any way (rc_this_oldid == 0 for these entries). It also doesn't use the newly added fields in any way. I intend to address the concerns mentioned above in a new patch, but in the meantime, some review would be nice. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 17237] Better integration of patrol feature
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 --- Comment #3 from Roan Kattouw 2009-01-30 11:37:15 UTC --- (In reply to comment #2) > I just looked at bug 17215 again and there's talk of adding an index, > something > I have only a very vague understanding of, but I assume has the same intention > of making revision/recentchanges queries easier. I guess that would be easier > than a schema change. > The basic idea of that index is that it makes it possible to retrieve the recentchanges entry for a certain revision efficiently, which semi-fixes this. Adding the bot and patrol flags to the revisions table makes more sense IMO, since, as Aryeh said on that bug, recentchanges serves just to summarize other tables, and as such should not contain 'new' information (i.e. information that isn't present in any other table). -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 17237] Better integration of patrol feature
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 --- Comment #2 from Gurch 2009-01-29 23:29:44 UTC --- (In reply to comment #1) > I agree, let's put the patrol flag (and the bot flag too, for that matter) in > the revisions table as well. Bot flag kind of makes sense where it is, given that its purpose (at least its original purpose) is to keep bulk edits out of recentchanges. (The somewhat fuzzy relationship between bot accounts and bot edits always confuses me.) Though having said that, I guess patrol flag kind of makes sense where it is too, if its purpose is to review new edits. I just looked at bug 17215 again and there's talk of adding an index, something I have only a very vague understanding of, but I assume has the same intention of making revision/recentchanges queries easier. I guess that would be easier than a schema change. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l
[Bug 17237] Better integration of patrol feature
https://bugzilla.wikimedia.org/show_bug.cgi?id=17237 Roan Kattouw changed: What|Removed |Added CC||roan.katt...@home.nl --- Comment #1 from Roan Kattouw 2009-01-29 23:04:25 UTC --- I agree, let's put the patrol flag (and the bot flag too, for that matter) in the revisions table as well. -- Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the assignee for the bug. You are on the CC list for the bug. ___ Wikibugs-l mailing list Wikibugs-l@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikibugs-l