Would you please review the attached patch which adds code to the flickr module to check for flickr users that should be denied the opportunity of rendering their tagged public photos on spreadfirefox This patch would need to be pushed later with the following table .. CREATE TABLE `flickr_denied` ( `nsid` varchar(64) NOT NULL ) ENGINE=MyISAM DEFAULT CHARSET=latin1;
Created attachment 318608 [details] [diff] [review] flickr_block.module patch
Attachment #318608 - Flags: review?
Does the patch let admins disable specific pictures or specific tags or both?
It allows administrators to block a flickr user. So if a spreadfirefox user posts something on flickr tagged with spreadfirefox that is inappropriate we can add the flickr id to the flickr_denied table and block the user from showing all images on spreadfirefox . What we have with this patch is the minimum required to solve the problem. When this is pushed i can work on extending this so we can block the image(s) only and provide some GUI interface to handle all of this Paul Booker
Over to webdev for review.
Assignee: server-ops → nobody
Component: Server Operations → Webdev
QA Contact: justin → webdev
sorting it out...
Assignee: nobody → paul
Component: Webdev → spreadfirefox.com
Priority: -- → P1
Product: mozilla.org → Websites
Version: other → unspecified
Comment on attachment 318608 [details] [diff] [review] flickr_block.module patch patch looks good. tested it. Need to update database as indicated above to get this to work
Attachment #318608 - Flags: review?(cpollett) → review+
Paul, please commit this. Do you have access to run the SQL on stage? If so, do that first, then commit the patch. If not, let me know. After we deploy it to stage, we can ask IT to push it live.
Alex , I have added the SQL statement with the patch and commited to subversion. If we can get the changes pushed live ill take it from there. Thanks Paul
Created attachment 318900 [details] [diff] [review] flickr_block patch
Attachment #318608 - Attachment is obsolete: true
Hi Paul, I think you checked this in. Should we mark it as resolved?
Hi Chris I'll resolve this as bug fixed when the changes have been pushed to production and i have updated the database successfully Thanks Paul
Hi Chris, Has this patch been pushed to production and the database schema updated on the status report page ? If so would you close this ticket please. Thanks Paul
Assignee: paul → cpollett
yes, it has been pushed and the db is update.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Thanks Alex :-)
You need to log in before you can comment on or make changes to this bug.