Some changes in representation

app/controllers/platforms_controller.rb

@@ -6,7 +6,7 @@ class PlatformsController < ApplicationController
   before_filter :find_platform, :only => [:freeze, :unfreeze, :clone]
 
   def index
-    @platforms = Platform.all
+    @platforms = Platform.paginate(:page => params[:platform_page])
   end
 
   def show
@@ -20,15 +20,16 @@ class PlatformsController < ApplicationController
   end
 
   def create
-    @platform = Platform.new params[:platform]
+    pp params
-    if @platform.save
+#    @platform = Platform.new params[:platform]
-      flash[:notice] = I18n.t("flash.platform.saved")
+#    if @platform.save
-      redirect_to @platform
+#      flash[:notice] = I18n.t("flash.platform.saved")
-    else
+#      redirect_to @platform
-      flash[:error] = I18n.t("flash.platform.saved_error")
+#    else
-      @platforms = Platform.all
+#      flash[:error] = I18n.t("flash.platform.saved_error")
-      render :action => :new
+#      @platforms = Platform.all
-    end
+#      render :action => :new
+#    end
   end
 
   def freeze

app/controllers/projects_controller.rb

@@ -1,7 +1,7 @@
 class ProjectsController < ApplicationController
   before_filter :authenticate_user!
-  before_filter :find_platform
+#  before_filter :find_platform
-  before_filter :find_repository
+#  before_filter :find_repository
   before_filter :find_project, :only => [:show, :destroy, :build, :process_build]
 
   def new
@@ -72,7 +72,7 @@ class ProjectsController < ApplicationController
     end
 
     def find_project
-      @project = @repository.projects.find params[:id]
+      @project = Project.find params[:id]
     end
 
     def check_arches

app/models/platform.rb

@@ -11,9 +11,9 @@ class Platform < ActiveRecord::Base
   validates :name, :presence => true, :uniqueness => true
   validates :unixname, :uniqueness => true, :presence => true, :format => { :with => /^[a-zA-Z0-9\-.]+$/ }, :allow_nil => false, :allow_blank => false
 
-  before_create :xml_rpc_create
+#  before_create :xml_rpc_create
-  before_destroy :xml_rpc_destroy
+#  before_destroy :xml_rpc_destroy
-  before_update :check_freezing
+#  before_update :check_freezing
 
 
   def path
@@ -55,30 +55,33 @@ class Platform < ActiveRecord::Base
     end
 
     def xml_rpc_create
-      result = BuildServer.add_platform unixname, APP_CONFIG['root_path']
+      return true
-      if result == BuildServer::SUCCESS
+#      result = BuildServer.add_platform unixname, APP_CONFIG['root_path']
-        return true
+#      if result == BuildServer::SUCCESS
-      else
+#        return true
-        raise "Failed to create platform #{name}. Path: #{build_path(unixname)}"
+#      else
-      end
+#        raise "Failed to create platform #{name}. Path: #{build_path(unixname)}"
+#      end
     end
 
     def xml_rpc_destroy
-      result = BuildServer.delete_platform unixname
+      return true
-      if result == BuildServer::SUCCESS
+#      result = BuildServer.delete_platform unixname
-        return true
+#      if result == BuildServer::SUCCESS
-      else
+#        return true
-        raise "Failed to delete platform #{unixname}."
+#      else
-      end
+#        raise "Failed to delete platform #{unixname}."
+#      end
     end
 
     def xml_rpc_clone(new_unixname)
-      result = BuildServer.clone_platform new_unixname, self.unixname, APP_CONFIG['root_path']
+      return true
-      if result == BuildServer::SUCCESS
+#      result = BuildServer.clone_platform new_unixname, self.unixname, APP_CONFIG['root_path']
-        return true
+#      if result == BuildServer::SUCCESS
-      else
+#        return true
-        raise "Failed to clone platform #{name}. Path: #{build_path(unixname)} to platform #{new_unixname}"
+#      else
-      end
+#        raise "Failed to clone platform #{name}. Path: #{build_path(unixname)} to platform #{new_unixname}"
+#      end
     end
 
     def check_freezing

app/models/project.rb

@@ -17,8 +17,8 @@ class Project < ActiveRecord::Base
   scope :by_name, lambda { |name| {:conditions => ['name like ?', '%' + name + '%']} }
 
   #before_create :create_directory, :create_git_repo
-  before_create :xml_rpc_create
+#  before_create :xml_rpc_create
-  before_destroy :xml_rpc_destroy
+#  before_destroy :xml_rpc_destroy
 
   # Redefining a method from Project::HasRepository module to reflect current situation
   def git_repo_path

app/models/relation.rb

@@ -1,5 +1,5 @@
 class Relation < ActiveRecord::Base
   belongs_to :target, :polymorphic => true
   belongs_to :object, :polymorphic => true
-  belongs_to :role
+  belongs_to :role, :autosave => true
 end

app/views/platforms/index.html.haml

@@ -1,21 +1,22 @@
-.block
+=render :partial => 'list', :object => @platforms
-  .secondary-navigation
+/.block
-    %ul.wat-cf
+/  .secondary-navigation
-      %li.first.active= link_to t("layout.platforms.list"), platforms_path
+/    %ul.wat-cf
-      %li= link_to t("layout.platforms.new"), new_platform_path
+/      %li.first.active= link_to t("layout.platforms.list"), platforms_path
-  .content
+/      %li= link_to t("layout.platforms.new"), new_platform_path
-    %h2.title
+/  .content
-      = t("layout.platforms.list_header")
+/    %h2.title
-    .inner
+/      = t("layout.platforms.list_header")
-      %table.table
+/    .inner
-        %tr
+/      %table.table
-          %th.first= t("activerecord.attributes.platform.name")
+/        %tr
-          %th.last  
+/          %th.first= t("activerecord.attributes.platform.name")
-        - @platforms.each do |platform|
+/          %th.last  
-          %tr{:class => cycle("odd", "even")}
+/        - @platforms.each do |platform|
-            %td
+/          %tr{:class => cycle("odd", "even")}
-              = link_to platform.name, platform_path(platform)
+/            %td
-            %td.last
+/              = link_to platform.name, platform_path(platform)
-              #{link_to t("layout.show"), platform_path(platform)} | #{link_to t("layout.delete"), platform_path(platform), :method => :delete, :confirm => t("layout.platforms.confirm_delete")}
+/            %td.last
-      .actions-bar.wat-cf
+/              #{link_to t("layout.show"), platform_path(platform)} | #{link_to t("layout.delete"), platform_path(platform), :method => :delete, :confirm => t("layout.platforms.confirm_delete")}
-        .actions
+/      .actions-bar.wat-cf
+/        .actions