Declaring our variables as for https://drupal.org/project/variable would also imply support for https://drupal.org/project/domain.

Initially we would like to avoid to add a dependency on the Variable module, thus only a patch for the latest stable version would be posted here for those that seek this functionality.

Watching the number of followers. Follow if you are interested.

Comments

lolandese’s picture

Issue summary: View changes
lolandese’s picture

Version: 7.x-1.5 » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new23.32 KB

Attached untested patch:

  • replaces some instances of variable_get() with variable_get_value() for the Flickr main module (see below)
  • provides an update function to delete all unused variables on running update.php and removes them from the uninstall hook
  • defines the variables as domain_variables.

Only affects the following variables (machine names):

  • flickr_default_userid
  • flickr_photos_per_page
  • flickr_title_suppress_on_small
  • flickr_metadata_suppress_on_small
  • flickr_info_overlay
  • flickr_license
lolandese’s picture

StatusFileSize
new21.58 KB

Correction. Remains untested.

lolandese’s picture

Removed one variable too much:

-  variable_del('flickr_caption_padding');