merb_has_flash 0.9.2 → 0.9.6

data/README

@@ -11,4 +11,5 @@ Hacked together by Michael Ivey <ivey@gweezlebur.com> ... send bug
 reports and patches to him.
 
 Contributors:
-Tim Kofol
+Tim Kofol
+Shay Arnett

data/Rakefile

@@ -1,30 +1,7 @@
 require 'rubygems'
 require 'rake/gempackagetask'
 
-PLUGIN = "merb_has_flash"
-NAME = "merb_has_flash"
-VERSION = "0.9.2"
-AUTHOR = "Michael Ivey"
-EMAIL = "ivey@gweezlebur.com"
-HOMEPAGE = "http://merb-plugins.rubyforge.org/merb_has_flash/"
-SUMMARY = "Merb plugin that provides a Rails-style flash"
-
-spec = Gem::Specification.new do |s|
-  s.name = NAME
-  s.version = VERSION
-  s.platform = Gem::Platform::RUBY
-  s.has_rdoc = true
-  s.extra_rdoc_files = ["README", "LICENSE", 'TODO']
-  s.summary = SUMMARY
-  s.description = s.summary
-  s.author = AUTHOR
-  s.email = EMAIL
-  s.homepage = HOMEPAGE
-  s.add_dependency('merb-core', '>= 0.9.2')
-  s.require_path = 'lib'
-  s.autorequire = PLUGIN
-  s.files = %w(LICENSE README Rakefile TODO) + Dir.glob("{lib,specs}/**/*")
-end
+spec = eval(File.read('merb_has_flash.gemspec'))
 
 Rake::GemPackageTask.new(spec) do |pkg|
   pkg.gem_spec = spec
@@ -36,4 +13,4 @@ end
 
 task :release => :package do
   sh %{rubyforge add_release merb-plugins merb_has_flash #{VERSION} pkg/#{NAME}-#{VERSION}.gem}
-end
+end

data/lib/merb_has_flash.rb

@@ -4,16 +4,18 @@ end
 
 require 'merb_has_flash/flash_hash'
 
+require 'merb_has_flash/helper'
+Merb::RenderMixin.send :include, MerbHasFlash::FlashHelperMixin
+
 require 'merb_has_flash/controller_extension'
 Merb::Controller.send :include, MerbHasFlash::ControllerExtension
 
-require 'merb_has_flash/helper'
-Merb::RenderMixin.send :include, MerbHasFlash::FlashHelperMixin
+
 
 module MerbHasFlash
   # The flash provides a way to pass temporary objects between actions. Anything you place in the flash will be exposed
   # to the very next action and then cleared out. This is a great way of doing notices and alerts, such as a create action
-  # that sets <tt>flash[:notice] = "Successfully created"</tt> before redirecting to a display action that can then expose 
+  # that sets <tt>flash[:notice] = "Successfully created"</tt> before redirecting to a display action that can then expose
   # the flash to its template. Actually, that exposure is automatically done. Example:
   #
   #   class WeblogController < Merb::Controller

data/lib/merb_has_flash/controller_extension.rb

@@ -2,36 +2,19 @@ module MerbHasFlash
   module ControllerExtension
     def self.included(base)
       base.send :include, InstanceMethods
-      
-      base.class_eval {
-        after :sweep_flash
-      }
+      base.class_eval { after :sweep_flash }
     end
-    
+
     module InstanceMethods
-      # Access the contents of the flash. Use <tt>flash["notice"]</tt> to read a notice you put there or 
-      # <tt>flash["notice"] = "hello"</tt> to put a new one.
-      # Note that if sessions are disabled only flash.now will work.
-      def flash(refresh = false) #:doc:
-        if !defined?(@_flash) || refresh
-          @_flash =
-            if session.is_a?(Hash)
-              # don't put flash in session if disabled
-              FlashHash.new
-            else
-              # otherwise, session is a CGI::Session or a TestSession
-              # so make sure it gets retrieved from/saved to session storage after request processing
-              session["flash"] ||= FlashHash.new
-            end
-        end
 
-        @_flash
+      def flash(refresh = false) #:doc:
+        session['flash'] = FlashHash.new unless refresh || session['flash'].is_a?(FlashHash)
+        session['flash']
       end
-      
-      protected
+
+    protected
       def sweep_flash
-        flash.sweep if request.session
-        session["flash"] = flash
+        flash.sweep
       end
     end
   end

data/lib/merb_has_flash/flash_hash.rb

@@ -3,45 +3,55 @@ module MerbHasFlash
     def initialize(flash)
       @flash = flash
     end
-    
-    def []=(k, v)
-      @flash[k] = v
-      @flash.discard(k)
-      v
+
+    def []=(key, val)
+      @flash[key] = val
+      @flash.discard key
+      val
     end
-    
-    def [](k)
-      @flash[k]
+
+    def [](key)
+      @flash[key]
     end
   end
-  
-  class FlashHash < Hash
-    def initialize #:nodoc:
-      super
-      @used = {}
+
+  class FlashHash
+    def initialize(*args) #:nodoc:
+      @attrs = Hash.new(*args)
+      @keepers = []
     end
-    
-    def []=(k, v) #:nodoc:
-      keep(k)
-      super
+
+    def []=(key, val) #:nodoc:
+      @attrs[key] = val
+      keep key
     end
-    
-    def update(h) #:nodoc:
-      h.keys.each{ |k| discard(k) }
-      super
+
+    def [](key)
+      @attrs[key]
     end
-    
+
+    def update(hash) #:nodoc:
+      @attrs.update hash
+      hash.keys.each { |key| keep key }
+    end
+
     alias :merge! :update
-    
-    def replace(h) #:nodoc:
-      @used = {}
-      super
+
+    def replace(hash) #:nodoc:
+      @attrs.replace hash
+      keep  # keep with no args automatically clears out unused keys and keeps all used ones.
+    end
+
+
+    def method_missing(method_name, *args) #:nodoc:
+      @attrs.send(method_name, *args)
     end
-  
+
+
     # Sets a flash that will not be available to the next action, only to the current.
     #
     #     flash.now[:message] = "Hello current action"
-    # 
+    #
     # This method enables you to use the flash as a central messaging system in your app.
     # When you need to pass an object to the next action, you use the standard flash assign (<tt>[]=</tt>).
     # When you need to pass an object to the current action, you use <tt>now</tt>, and your object will
@@ -49,53 +59,33 @@ module MerbHasFlash
     #
     # Entries set via <tt>now</tt> are accessed the same way as standard entries: <tt>flash['my-key']</tt>.
     def now
-      FlashNow.new self
+      @_fn ||= FlashNow.new self  # This way, a new object is not created on every "now."
     end
-  
+
     # Keeps either the entire current flash or a specific flash entry available for the next action:
     #
     #    flash.keep            # keeps the entire flash
-    #    flash.keep(:notice)   # keeps only the "notice" entry, the rest of the flash is discarded
-    def keep(k = nil)
-      use(k, false)
+    #    flash.keep(:notice)   # keeps only the "notice" entry
+    def keep(key = nil)
+      key.nil? ? @keepers = @attrs.keys : @keepers << key unless @keepers.include?(key)
     end
-  
+
     # Marks the entire flash or a single flash entry to be discarded by the end of the current action
     #
-    #     flash.keep                 # keep entire flash available for the next action
-    #     flash.discard(:warning)    # discard the "warning" entry (it'll still be available for the current action)
-    def discard(k = nil)
-      use(k)
+    #     flash.discard              # discard the entire hash (it'll still be available for the current action)
+    #     flash.discard(:warning)    # discard the "warning" entry (still available as above)
+    def discard(key = nil)
+      key.nil? ? @keepers = [] : @keepers.delete(key)
     end
-  
-    # Mark for removal entries that were kept, and delete unkept ones.
-    #
+
+    # Clear the keys that are kept, and delete the ones that are currently unkept
     # This method is called automatically by filters, so you generally don't need to care about it.
     def sweep #:nodoc:
-      keys.each do |k| 
-        unless @used[k]
-          use(k)
-        else
-          delete(k)
-          @used.delete(k)
-        end
-      end
-
-      (@used.keys - keys).each{|k| @used.delete k } # clean up after keys that could have been left over by calling reject! or shift on the flash
+      @attrs.keys.each { |key| @attrs.delete key unless @keepers.include?(key) }
+      discard
     end
-  
-    private
-      # Used internally by the <tt>keep</tt> and <tt>discard</tt> methods
-      #     use()               # marks the entire flash as used
-      #     use('msg')          # marks the "msg" entry as used
-      #     use(nil, false)     # marks the entire flash as unused (keeps it around for one more action)
-      #     use('msg', false)   # marks the "msg" entry as unused (keeps it around for one more action)
-      def use(k=nil, v=true)
-        unless k.nil?
-          @used[k] = v
-        else
-          keys.each{|key| use key, v }
-        end
-      end
+
+
   end
-end
+
+end

data/spec/merb_has_flash/controller_extension_spec.rb

@@ -0,0 +1,5 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe "merb_has_flash Controller Extension" do
+  it "should have specs, but doesn't..."
+end

data/spec/merb_has_flash/flash_hash_spec.rb

@@ -0,0 +1,47 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+require 'merb_has_flash/flash_hash'
+
+describe "merb_has_flash FlashHash" do
+  before(:each) do
+    @hash = MerbHasFlash::FlashHash.new
+  end
+
+  it "should not be a kind of Hash" do
+    # Merb 0.9.6 now transmutes all Hashes in sessions into Mashes.. destroying the functionality.
+    @hash.should_not be_a_kind_of(Hash)
+  end
+
+  it "should keep appropriate variables on sweep" do
+    @hash[:a] = true
+    @hash[:b] = false
+    @hash[:c] = 90928
+
+    @hash.sweep
+
+    @hash[:a].should eql(true)
+    @hash[:b].should eql(false)
+    @hash[:c].should eql(90928)
+  end
+
+  it "should not keep variables when using now" do
+    @hash.now[:a] = true
+    @hash[:b] = true
+    @hash.instance_variable_get(:@keepers).should_not include(:a)
+  end
+
+  it "should discard appropriate variables on sweep" do
+    @hash.now[:a] = true
+    @hash[:b] = false
+    @hash[:c] = 90928
+    @hash.discard(:c)
+    @hash[:d] = 'Sir, the smurfs are eating your lunch...'
+
+    @hash.sweep
+
+    @hash[:a].should be_nil
+    @hash[:b].should eql(false)
+    @hash[:c].should be_nil
+    @hash[:d].should  eql('Sir, the smurfs are eating your lunch...')
+  end
+
+end

data/spec/spec_helper.rb

@@ -0,0 +1,2 @@
+$TESTING=true
+$:.push File.join(File.dirname(__FILE__), '..', 'lib')

metadata

@@ -1,48 +1,50 @@
 --- !ruby/object:Gem::Specification 
 name: merb_has_flash
 version: !ruby/object:Gem::Version 
-  version: 0.9.2
+  version: 0.9.6
 platform: ruby
 authors: 
-- Michael Ivey
-autorequire: merb_has_flash
+- Michael D. Ivey
+- Jeremy Nicoll
+autorequire: 
 bindir: bin
 cert_chain: []
 
-date: 2008-03-27 00:00:00 -05:00
+date: 2008-09-10 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: merb-core
+  type: :runtime
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
-    - - ">="
+    - - ">"
       - !ruby/object:Gem::Version 
-        version: 0.9.2
+        version: 0.9.0
     version: 
-description: Merb plugin that provides a Rails-style flash
+description: Rails' 'flash' session notification system ported to Merb
 email: ivey@gweezlebur.com
 executables: []
 
 extensions: []
 
-extra_rdoc_files: 
-- README
-- LICENSE
-- TODO
+extra_rdoc_files: []
+
 files: 
 - LICENSE
-- README
 - Rakefile
+- README
 - TODO
-- lib/merb_has_flash
+- lib/merb_has_flash.rb
 - lib/merb_has_flash/controller_extension.rb
 - lib/merb_has_flash/flash_hash.rb
 - lib/merb_has_flash/helper.rb
-- lib/merb_has_flash.rb
-has_rdoc: true
-homepage: http://merb-plugins.rubyforge.org/merb_has_flash/
+- spec/merb_has_flash/controller_extension_spec.rb
+- spec/merb_has_flash/flash_hash_spec.rb
+- spec/spec_helper.rb
+has_rdoc: false
+homepage: http://github.com/ivey/merb_has_flash
 post_install_message: 
 rdoc_options: []
 
@@ -63,9 +65,11 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.0.1
+rubygems_version: 1.2.0
 signing_key: 
 specification_version: 2
-summary: Merb plugin that provides a Rails-style flash
-test_files: []
-
+summary: Rails' 'flash' session notification system ported to Merb
+test_files: 
+- spec/merb_has_flash/controller_extension_spec.rb
+- spec/merb_has_flash/flash_hash_spec.rb
+- spec/spec_helper.rb