aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--app/controllers/admin_public_body_controller.rb6
-rw-r--r--app/models/public_body.rb3
2 files changed, 5 insertions, 4 deletions
diff --git a/app/controllers/admin_public_body_controller.rb b/app/controllers/admin_public_body_controller.rb
index 79fa98475..a43be17ce 100644
--- a/app/controllers/admin_public_body_controller.rb
+++ b/app/controllers/admin_public_body_controller.rb
@@ -4,7 +4,7 @@
# Copyright (c) 2007 UK Citizens Online Democracy. All rights reserved.
# Email: francis@mysociety.org; WWW: http://www.mysociety.org/
#
-# $Id: admin_public_body_controller.rb,v 1.7 2008-02-21 16:15:45 francis Exp $
+# $Id: admin_public_body_controller.rb,v 1.8 2008-03-02 23:46:51 francis Exp $
class AdminPublicBodyController < ApplicationController
layout "admin"
@@ -54,7 +54,9 @@ class AdminPublicBodyController < ApplicationController
end
def destroy
- PublicBody.find(params[:id]).destroy
+ public_body = PublicBody.find(params[:id])
+ public_body.tag_string = ""
+ public_body.destroy
flash[:notice] = "PublicBody was successfully destroyed."
redirect_to admin_url('body/list')
end
diff --git a/app/models/public_body.rb b/app/models/public_body.rb
index 12fe880c2..c92972df1 100644
--- a/app/models/public_body.rb
+++ b/app/models/public_body.rb
@@ -22,7 +22,7 @@
# Copyright (c) 2007 UK Citizens Online Democracy. All rights reserved.
# Email: francis@mysociety.org; WWW: http://www.mysociety.org/
#
-# $Id: public_body.rb,v 1.28 2008-03-02 23:43:09 francis Exp $
+# $Id: public_body.rb,v 1.29 2008-03-02 23:46:51 francis Exp $
require 'csv'
require 'set'
@@ -134,7 +134,6 @@ class PublicBody < ActiveRecord::Base
else
# New public body
public_body = PublicBody.new(:name => name, :request_email => email, :complaint_email => "", :short_name => "", :last_edit_editor => "import_csv", :last_edit_comment => 'Created from spreadsheet')
- public_body.save! # XXX shouldn't need this save, but without it the PublicBodyTag doesn't validate as no PublicBody id, and there is no harm cause we're in a transaction
public_body.tag_string = tag
public_body.save!