From: Tomas Sedovic <[email protected]>
This new page is capable of viewing/updating/creating new providers, so the
funtionality has been moved there.
---
src/app/controllers/provider_controller.rb | 24 +++++++++++++
src/app/views/provider/_providers.haml | 19 +++++++---
src/app/views/provider/show.haml | 52 +++++++++++++++++++++------
src/config/locales/en.yml | 13 +++++++
src/config/navigation.rb | 2 +-
5 files changed, 92 insertions(+), 18 deletions(-)
diff --git a/src/app/controllers/provider_controller.rb
b/src/app/controllers/provider_controller.rb
index af8c3f9..eb8f34c 100644
--- a/src/app/controllers/provider_controller.rb
+++ b/src/app/controllers/provider_controller.rb
@@ -27,14 +27,24 @@ class ProviderController < ApplicationController
end
def show
+ @providers = Provider.list_for_user(@current_user,
Privilege::PROVIDER_VIEW)
@provider = Provider.find(:first, :conditions => {:id => params[:id]})
require_privilege(Privilege::PROVIDER_VIEW, @provider)
end
+ def edit
+ @providers = Provider.list_for_user(@current_user,
Privilege::PROVIDER_MODIFY)
+ @provider = Provider.find(:first, :conditions => {:id => params[:id]})
+ require_privilege(Privilege::PROVIDER_MODIFY, @provider)
+ render :show
+ end
+
def new
require_privilege(Privilege::PROVIDER_MODIFY)
+ @providers = Provider.list_for_user(@current_user,
Privilege::PROVIDER_MODIFY)
@provider = Provider.new(params[:provider])
condormatic_classads_sync
+ render :show
end
def create
@@ -50,6 +60,20 @@ class ProviderController < ApplicationController
condormatic_classads_sync
end
+ def update
+ require_privilege(Privilege::PROVIDER_MODIFY)
+ @provider = Provider.find(:first, :conditions => {:id =>
params[:provider][:id]})
+ @provider.name = params[:provider][:name]
+
+ if @provider.save
+ flash[:notice] = "Provider updated."
+ redirect_to :action => "show", :id => @provider
+ else
+ render :action => "edit"
+ end
+ condormatic_classads_sync
+ end
+
def destroy
if request.post?
@provider = Provider.find(params[:provider][:id])
diff --git a/src/app/views/provider/_providers.haml
b/src/app/views/provider/_providers.haml
index 4b8008e..47a680a 100644
--- a/src/app/views/provider/_providers.haml
+++ b/src/app/views/provider/_providers.haml
@@ -1,7 +1,16 @@
%h2 Providers
%ul#providers
- - @providers.each do |provider|
- %li
- = link_to provider.name, {:controller => "provider", :action => "show",
:id => provider }, :class => "providername"
-- if
BasePermissionObject.general_permission_scope.has_privilege(@current_user,
Privilege::PROVIDER_MODIFY)
- = link_to "Add a provider", {:controller => "provider", :action => "new"},
:class => "actionlink"
+#providers_nav.grid_3
+ %dl
+ %dt
+ = t('provider.providers')
+ - @providers.each do |provider|
+ %dd
+ %span
+ - selected = 'selected' if @provider and (provider.id == @provider.id)
+ %a{ :href => url_for(:controller => 'provider', :action => 'show', :id
=> provider), :class => selected }
+ = provider.name
+ - form_tag :controller => 'provider', :action => 'edit', :id => @provider
do
+ = submit_tag t(:edit), :disabled => ('disabled' unless @provider and
controller.action_name == 'show')
+ - form_tag({:controller => 'provider', :action => 'new'}, {:method => :get
}) do
+ %input{ :type => 'submit', :value => t(:add), :disabled => ('disabled'
if @providers.length > 1) }
diff --git a/src/app/views/provider/show.haml b/src/app/views/provider/show.haml
index b3e0bd7..81726e7 100644
--- a/src/app/views/provider/show.haml
+++ b/src/app/views/provider/show.haml
@@ -1,12 +1,40 @@
-- content_for :scripts do
- :javascript
- $(document).ready(function() {
- $("#provider-tabs").tabs();
- });
-#provider-tabs
- %ul
- %li= link_to "Accounts", {:action => "accounts", :id => @provider.id,
:ajax => true}
- %li= link_to "Realms", {:action => "realms", :id => @provider.id, :ajax
=> true }
- - if has_view_perms?
- %li= link_to "User access", {:controller => "permissions", :action =>
"list", :provider_id => @provider.id, :ajax => true }
- %li= link_to "Settings", {:action => "settings", :id => @provider.id,
:ajax => true}
+- readonly = controller.action_name == 'show' ? true : false
+= render :partial => 'providers'
+#details.grid_13
+ = render_navigation(:level => 4)
+ - if controller.action_name == 'new'
+ - form_action = 'create'
+ - elsif controller.action_name == 'edit'
+ - form_action = 'update'
+ - form_tag :controller => :provider, :action => form_action do
+ %h2.first
+ = t('.provider_name')
+ - unless readonly
+ %span.required
+ *
+ %h2.second
+ = t('.provider_url')
+ - unless readonly
+ %span.required
+ *
+ .fieldGroup.clearfix
+ = text_field :provider, :name, :title => t('.provider_name'), :value =>
(@provider.name if @provider), :disabled => ('disabled' if
controller.action_name == 'show')
+ = text_field :provider, :url, :title => t('.provider_url'), :class =>
'emailinput', :value => (@provider.url if @provider), :disabled => ('disabled'
unless controller.action_name == 'new')
+ - if controller.action_name == 'edit':
+ = hidden_field :provider, :id, :value => @provider.id
+ %p
+ %span
+ (
+ %a{ :href => ''}<>
+ = t('.test_connection')
+ )
+ - unless readonly
+ %p.requirement
+ %span.required
+ *
+ \-
+ = t('.required_field')
+ - if controller.action_name == 'edit'
+ %input{ :type => 'submit', :value => t(:save), :name => 'save_provider',
:id => 'save_provider' }
+ - elsif controller.action_name == 'new'
+ %input{ :type => 'submit', :value => t(:add), :name => 'add_provider',
:id => 'add_provider' }
diff --git a/src/config/locales/en.yml b/src/config/locales/en.yml
index c435f00..3e6918c 100644
--- a/src/config/locales/en.yml
+++ b/src/config/locales/en.yml
@@ -39,6 +39,9 @@ en:
launch_instance: Launch Instance
help: Help
continue: Continue
+ edit: Edit
+ add: Add
+ save: Save
settings:
index:
general_settings: General Settings
@@ -55,3 +58,13 @@ en:
define_services_desc: View, edit and define services that DeltaCloud
will offer. These services will be mapped to individual providers based on
their capabilities.
permissions: Permissions
permissions_desc: Create and edit User Treatments that are applied to
users. Treatments pair together Roles with Categories of Instances, Pools and
Templates. Manage the categories, and define and edit Roles. Roles group
together sets of permissions that are relevant to Pools, Templates and
Instances.
+ provider:
+ providers: PROVIDERS
+ show:
+ provider_name: Provider Name
+ provider_url: Provider URL
+ caution_alt_text: Caution
+ enter_correct_url_msg: Please enter a correct URL format.
+ test_connection: Test Connection
+ required_field: Required field.
+ caution_image: Caution
diff --git a/src/config/navigation.rb b/src/config/navigation.rb
index 17e06ed..662462f 100644
--- a/src/config/navigation.rb
+++ b/src/config/navigation.rb
@@ -8,7 +8,7 @@ SimpleNavigation::Configuration.run do |navigation|
first_level.item :administration, t(:administration), '', :class =>
'administration' do |second_level|
second_level.item :system_settings, t(:system_settings), "/settings" do
|third_level|
third_level.item :manage_providers, t(:manage_providers), "/provider"
do |fourth_level|
- fourth_level.item :provider_summary, t(:provider_summary),
"/provider/show"
+ fourth_level.item :provider_summary, t(:provider_summary),
"/provider/show", :highlights_on => /\/provider\/(show|edit|new)/
fourth_level.item :provider_accounts, t(:provider_accounts),
"/provider/accounts"
end
third_level.item :self_service_settings, t(:self_service_settings),
"/settings/self-service"
--
1.7.2.3
_______________________________________________
deltacloud-devel mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/deltacloud-devel
