From e6a9555dd88d884492ef526ac27b6f261694dd82 Mon Sep 17 00:00:00 2001
From: Rueshyna <rueshyna(XD)gmail.com>
Date: Thu, 8 Nov 2012 12:34:26 +0800
Subject: [PATCH] refine the location module done

---
 .../gprs/back_end/locations_controller.rb     | 26 +++++++++----------
 .../gprs/back_end/locations/_form.html.erb    |  4 ---
 .../gprs/back_end/locations/edit.html.erb     |  4 ++-
 .../gprs/back_end/locations/index.html.erb    |  6 ++---
 .../gprs/back_end/locations/new.html.erb      |  4 ++-
 5 files changed, 22 insertions(+), 22 deletions(-)

diff --git a/vendor/built_in_modules/gprs/app/controllers/panel/gprs/back_end/locations_controller.rb b/vendor/built_in_modules/gprs/app/controllers/panel/gprs/back_end/locations_controller.rb
index 1aff6223..3a5a6ea4 100644
--- a/vendor/built_in_modules/gprs/app/controllers/panel/gprs/back_end/locations_controller.rb
+++ b/vendor/built_in_modules/gprs/app/controllers/panel/gprs/back_end/locations_controller.rb
@@ -1,10 +1,10 @@
 class Panel::Gprs::BackEnd::LocationsController < OrbitBackendController
 
-  before_filter :clean_values, :only => [:create, :update]
+  #before_filter :clean_values, :only => [:create, :update]
 
   before_filter :force_order_for_visitor,:only => [:index, :new, :edit, :delete]
   before_filter :force_order_for_user,:except => :get_locations
-  before_filter :for_app_sub_manager,:except => [:index, :new, :edit, :delete, :get_locations]
+  #before_filter :for_app_sub_manager,:except => [:index, :new, :edit, :delete, :get_locations]
 
   def index
     @locations = Location.all
@@ -28,7 +28,7 @@ class Panel::Gprs::BackEnd::LocationsController < OrbitBackendController
   end
 
   def show
-    @location = Location..find(params[:id])
+    @location = Location.find(params[:id])
     respond_to do |format|
       format.html
       format.json { render json: @location }
@@ -36,22 +36,22 @@ class Panel::Gprs::BackEnd::LocationsController < OrbitBackendController
   end
 
   def create
-    debugger
     @location = Location.new(params[:location])
-    if @location.save
-      format.html { redirect_to @location  }
-      format.json { render json: @location, status: :created, location: @location }
-    end
-    #render :action => "new"
+    @location.save!
+    redirect_to panel_gprs_back_end_locations_url
+  end
+
+  def update
+    @location = Location.find(params[:id])
+    @location.update_attributes(params[:location])
+    redirect_to panel_gprs_back_end_locations_url
   end
 
   def destroy
-    @location = Location..find(params[:id])
+    @location = Location.find(params[:id])
     @location.destroy
+    redirect_to panel_gprs_back_end_locations_url
 
-    respond_to do |format|
-      format.json { head :no_content }
-    end
   end
 
   def get_locations
diff --git a/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/_form.html.erb b/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/_form.html.erb
index efda0d9e..62e8b21b 100644
--- a/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/_form.html.erb
+++ b/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/_form.html.erb
@@ -1,4 +1,3 @@
-<%= form_for @location, :url => 'panel_gprs_back_end_locations_path' ,:class=>"form-horizontal" do |f| %>
   <div class="control-group">
     <label class="control-label" for="">Name</label>
     <div class="controls">
@@ -29,6 +28,3 @@
       <%= f.submit "Save", :class=>"btn" %>
     </div>
   </div>
-  <% end  %>
-<!-- </form> -->
-
diff --git a/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/edit.html.erb b/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/edit.html.erb
index 37afa805..82a59afa 100644
--- a/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/edit.html.erb
+++ b/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/edit.html.erb
@@ -1,2 +1,4 @@
-<%= render 'form' %>
+<%= form_for @location, :url=>  panel_gprs_back_end_location_path(@location), :html => { :class=>"form-horizontal"} do |f| %>
+  <%= render :partial => 'form', :locals => {:f => f}  %>
+<% end  %>
 
diff --git a/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/index.html.erb b/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/index.html.erb
index 78dd48a8..6731688d 100644
--- a/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/index.html.erb
+++ b/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/index.html.erb
@@ -18,11 +18,11 @@
         <td><%= location.longitude%></td>
         <td><%= location.latitude%></td>
         <td><%= location.description%></td>
-        <td><%= link_to 'Edit', edit_panel_gprs_back_end_locations_path(location) %></td>
-        <td><%= link_to 'Destroy', location, method: :delete %>
+        <td><%= link_to 'Edit', edit_panel_gprs_back_end_location_path(location) %></td>
+        <td><%= link_to 'Destroy', panel_gprs_back_end_location_path(location), method: :delete , :confirm => t(:sure?) %></td>
       </tr>
     <% end %>
   </tbody>
 </table>
 
-<td><%= link_to 'New', new_panel_gprs_back_end_locations_path %></td>
+<td><%= link_to 'New', new_panel_gprs_back_end_location_path %></td>
diff --git a/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/new.html.erb b/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/new.html.erb
index 2d343636..4f911d83 100644
--- a/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/new.html.erb
+++ b/vendor/built_in_modules/gprs/app/views/panel/gprs/back_end/locations/new.html.erb
@@ -1 +1,3 @@
-<%= render 'form' %>
+<%= form_for @location, :url=>  panel_gprs_back_end_locations_path, :html => { :class=>"form-horizontal"} do |f| %>
+  <%= render :partial => 'form', :locals => {:f => f} %>
+<% end  %>