Posted by aaronbauman on April 17, 2009 at 9:10pm
| Project: | Flickr |
| Version: | 6.x-1.x-dev |
| Component: | flickr (main module) |
| Category: | bug report |
| Priority: | normal |
| Assigned: | aaronbauman |
| Status: | closed (fixed) |
Issue Summary
at least in my instance, the function flickr_admin_settings_submit never fires, and if the user enters a Username instead of the esoteric User Id, the validation passes but all other functionality doesn't work since flickr_default_userid doesn't store the proper uid.
This patch adds a single line to the FAPI form function to make sure that the FAPI submit function fires.
| Attachment | Size |
|---|---|
| flickr.admin_.inc_.patch | 344 bytes |
Comments
#1
I could reproduce the issue and the patch indeed solves it.
One minor remaining issue is that
system_settings_form_submit()is called twice on submit,I attached an updated patch to solve that too.
#2
I ran into this and my solution was to enter the esoteric User Id manually into the Flickr settings form, rather than the username. It validates and gets stored and used by Flickr attach with no problems. (not a user friendly approach, but a non-patch solution for the moment)
Edit: This is, indeed, a workaround :)
#3
that's not a "solution", but a "workaround" ;-)
#4
Good catch. Thanks. Committed to DRUPAL-6--1.
#5
Automatically closed -- issue fixed for 2 weeks with no activity.