Merge dcaf074753 into 137d997793

app/views/layouts/_admin_sidebar_index.html.haml

@@ -23,6 +23,10 @@
             New Conference
   - if can? :index, User
     %hr
+    %li
+      = link_to(admin_organizations_path) do
+        %span.fa.fa-group
+        Organizations
     %li
       = link_to(admin_users_path) do
         %span.fa.fa-user
@@ -37,8 +41,3 @@
       = link_to(admin_revision_history_path) do
         %span.fa.fa-history
         Revision History
-    - if ENV['ORGANIZATIONS_ENABLED'] == 'true'
-      %li
-        = link_to(admin_organizations_path) do
-          %span.fa.fa-group
-          Organizations

app/views/layouts/_user_menu.html.haml

@@ -37,8 +37,12 @@
         = link_to(admin_conferences_path()) do
           %span.fa.fa-home
           Administration
-      - if can? :new, Conference.new
-        =link_to(new_admin_conference_path) do
+      - if (can? :new, Organization.new) && !Organization.any?
+        = link_to(new_admin_organization_path) do
+          %span.fa.fa-plus
+          New Organization
+      - if (can? :new, Conference.new) && Organization.any?
+        = link_to(new_admin_conference_path) do
           %span.fa.fa-plus
           New Conference
     -if @conference && @conference.id && can?(:show, @conference)

spec/features/user_menu_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+feature '_user_menu' do
+  let(:admin) { create(:admin) }
+  subject { page }
+  before { sign_in admin }
+
+  scenario 'when no organization is present', feature: true, js: true do
+    visit root_path
+    click_link admin.name
+
+    is_expected.to have_text('New Organization')
+    is_expected.to_not have_text('New Conference')
+  end
+
+  scenario 'when organization is present', feature: true, js: true do
+    create(:organization)
+    visit root_path
+    click_link admin.name
+
+    is_expected.to_not have_text('New Organization')
+    is_expected.to have_text('New Conference')
+  end
+end