bcms_settings 0.0.3 → 0.0.4

data/{README → README.markdown}

@@ -101,13 +101,13 @@ To delete values, call the delete method on the settings object:
     => #<Cms::Settings: bcms_s3 => {"account_id"=>"NEW_ID"}
 
 Keys can have almost any name, except for these:
-["inspect", "__send__", "delete", "instance_eval", "__metaclass__", "method_missing"]
+    inspect, __send__, delete, instance_eval, __metaclass__, method_missing
 
 
 ### Registering and deleting modules
 
-It is also possible to register and delete modules manually in addition to or
-as an alternative to caling Settings.synchronize.
+It is also possible to register and delete modules manually in addition or
+as an alternative to calling Settings.synchronize.
 
 To register modules:
 
@@ -119,21 +119,18 @@ then you can store, retrieve and delete arbitrary values:
     config.client = "Widgets INC"
     config.url = "http://example.com"
     config.sections = %w[A B C]
-
     config.url
     => "http://example.com
-
     config.delete("url")
-
     config.url
     => nil
 
-In reality, 'registering a module' only creates an object where
-to store values, so you can request sorage to the Settings module for
+In the background, what 'registering a module' does is create an object where
+to store values, so you can request storage to the Settings module for
 whatever porpose you like, povinding that:
 
-  1. The name you are trying to register is a valid BrowserCMS module
-  name and a valid Ruby method identifier, so this is valid:
+1. The name you are trying to register is a valid BrowserCMS module
+name and a valid Ruby method identifier, so this is valid:
 
       Cms::Settings.register("bcms_my_config")
 
@@ -141,8 +138,8 @@ whatever porpose you like, povinding that:
 
       Cms::Settings.register("My Config")
 
-  2. The name you are trying to register has not been registered
-  previously. Names passed to the register method must be unique.
+2. The name you are trying to register has not been previously registered.
+Names passed to the register method must be unique.
 
 
 To delete modules:

data/lib/bcms_settings/cms/settings.rb

@@ -39,8 +39,10 @@ module Cms
     # get a configuration object whether they need it or not.
 
     def synchronize
-      register_modules(installed_modules - registered_modules)
-      remove_modules(managed_modules - installed_modules)
+      if CmsModule.table_exists?
+        register_modules(installed_modules - registered_modules)
+        remove_modules(managed_modules - installed_modules)
+      end
     end
 
     # Retruns an array of module names the Cms::Settings module knows
@@ -176,18 +178,16 @@ module Cms
     end
 
     def register_modules(module_names, managed = true)
-      if CmsModule.table_exists?
-        module_names.each do |name|
-          verify_module_name(name)
-          begin
-            CmsModule.create!(:name => name.to_s,
-                              :settings => {},
-                              :cms_managed => managed)
-
-          rescue ActiveRecord::RecordInvalid
-            raise ModuleConfigurationExists,
-              "The module #{name} is already registered."
-          end
+      module_names.each do |name|
+        verify_module_name(name)
+        begin
+          CmsModule.create!(:name => name.to_s,
+                            :settings => {},
+                            :cms_managed => managed)
+
+        rescue ActiveRecord::RecordInvalid
+          raise ModuleConfigurationExists,
+            "The module #{name} is already registered."
         end
       end
     end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: bcms_settings
 version: !ruby/object:Gem::Version 
-  hash: 25
+  hash: 23
   prerelease: false
   segments: 
   - 0
   - 0
-  - 3
-  version: 0.0.3
+  - 4
+  version: 0.0.4
 platform: ruby
 authors: 
 - BrowserMedia
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-11-29 00:00:00 -06:00
+date: 2010-11-30 00:00:00 -06:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -42,7 +42,7 @@ extensions: []
 
 extra_rdoc_files: 
 - LICENSE.txt
-- README
+- README.markdown
 files: 
 - app/controllers/application_controller.rb
 - app/helpers/application_helper.rb
@@ -54,7 +54,7 @@ files:
 - lib/bcms_settings/routes.rb
 - rails/init.rb
 - LICENSE.txt
-- README
+- README.markdown
 - test/performance/browsing_test.rb
 - test/test_helper.rb
 - test/unit/cms_module_test.rb