Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fixes #5172 - typo SETTINGS[:location_enabled] instead of :locations_enabled causes default location select box not to be visible #1373

Closed
wants to merge 1 commit into from

Conversation

isratrade
Copy link
Member

No description provided.

@isratrade
Copy link
Member Author

@elobato, check your settings.yaml file. It seems the only way for your commit 070b83f to have worked in your local dev environment is if you had location_enabled rather that locations_enabled

@dLobatog
Copy link
Member

I had that setting, sorry. Can you change it to SETTING[:locations_enabled] as per @domcleal 's comment #1191 (diff) ?

@isratrade
Copy link
Member Author

The helper method show_location_tab? uses SETTINGS[:locations_enabled]

  def show_location_tab?
    SETTINGS[:locations_enabled] && User.current.allowed_to?(:view_locations)
  end

@dLobatog
Copy link
Member

Yeah, I used it at first too, but the reason why @domcleal didn't want to use the helper was that we don't want users to need special permissions (:view_locations) to change their own default taxonomies.

…locations_enabled causes default location select box not to be visible
@isratrade
Copy link
Member Author

re-committed. It makes senses not to have (:view_conditions) to change default taxonomies.

@dLobatog
Copy link
Member

Merged as 936bab0 , thanks @isratrade !

@dLobatog dLobatog closed this Apr 16, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants