Page MenuHomePhabricator

add populateCategories.php script to project
Closed, ResolvedPublic

Description

https://gist.github.com/atdt/5159480 should be checked in to the project, perhaps under scripts. Even if out of date or incomplete, it'll reduce the "bus factor" of the extension :)


Version: master
Severity: normal

Details

Reference
bz46299

Event Timeline

bzimport raised the priority of this task from to Normal.Nov 22 2014, 1:28 AM
bzimport set Reference to bz46299.

This is at least normal priority. I'm not just copying from there since I don't know if that's the latest version.

ori added a comment.Aug 29 2013, 7:26 AM

The version I have on fenari differs in one respect: I realized that sAdd with multiple items wasn't working, and I changed it to iterate through $pages and call sAdd on each one. Now that PHPREDIS has been updated for Redis 2.6, this should not be necessary, but it'd be good to test.

tin is running Redis 2.6.3; phpredis is at version 2.2.3. I confirmed sAdd with variable arguments worked, so I'll just add the gist version.

Change 106479 had a related patch set uploaded by Mattflaschen:
Import populate script from Gist

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

Change 106479 merged by jenkins-bot:
Import populate script from Gist

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