radiant-reader-extension 3.0.32 → 3.0.33

data/app/controllers/accounts_controller.rb

@@ -152,7 +152,7 @@ private
 
   def get_readers_and_groups
     @readers = Reader.visible_to(current_reader)
-    @groups = current_reader.all_visible_groups
+    @groups = current_reader.groups
     @reader = Reader.find(params[:id]) if params[:id]
   end
 

data/app/controllers/groups_controller.rb

@@ -24,7 +24,7 @@ class GroupsController < ReaderActionController
 private
   
   def get_group_or_groups
-    @groups = current_reader.all_visible_groups
+    @groups = current_reader.groups
     @group = Group.find(params[:id]) if params[:id]
   end
 

data/app/models/group.rb

@@ -2,7 +2,6 @@ require 'ancestry'
 
 class Group < ActiveRecord::Base
 
-  has_ancestry
   belongs_to :leader, :class_name => 'Reader'
   belongs_to :created_by, :class_name => 'User'
   belongs_to :updated_by, :class_name => 'User'
@@ -66,7 +65,7 @@ class Group < ActiveRecord::Base
     when 'private'
       reader ? self.all : self.none
     when 'grouped'
-      (reader && reader.is_grouped?) ? reader.all_visible_groups : self.none
+      (reader && reader.is_grouped?) ? reader.groups : self.none
     else
       self.none
     end
@@ -75,24 +74,11 @@ class Group < ActiveRecord::Base
   def visible_to?(reader=nil)
     self.class.visible_to(reader).map(&:id).include? self.id
   end
-
-  def tree
-    # can't quite do this in one step, but we can return a scope
-    self.root.subtree
-  end
-  
-  def tree_ids
-    self.root.subtree_ids
-  end
   
   def members
-    Reader.in_groups(subtree)
+    readers
   end
   
-  def inherited_permissions
-    Permission.to_groups(path)
-  end
-    
   def url
     homepage.url if homepage
   end
@@ -130,7 +116,7 @@ class Group < ActiveRecord::Base
   def self.define_retrieval_methods(classname)
     type_scope = "for_#{classname.downcase.pluralize}".intern
     Permission.send :named_scope, type_scope, :conditions => { :permitted_type => classname }
-    define_method("#{classname.downcase}_permissions") { self.inherited_permissions.send type_scope }
+    define_method("#{classname.downcase}_permissions") { self.permissions.send type_scope }
     define_method("#{classname.downcase.pluralize}") {
       ids = self.send("#{classname.to_s.downcase}_permissions".intern).map(&:permitted_id)
       classname.constantize.find_these(ids)

data/app/models/reader.rb

@@ -93,7 +93,7 @@ class Reader < ActiveRecord::Base
     when 'private'
       reader ? self.all : self.none
     when 'grouped'
-      reader ? self.in_groups(reader.all_visible_group_ids) : self.none
+      reader ? self.in_groups(reader.group_ids) : self.none
     else
       self.none
     end
@@ -103,38 +103,12 @@ class Reader < ActiveRecord::Base
     (reader && (reader == self)) || self.class.visible_to(reader).map(&:id).include?(self.id)
   end
   
-  # returns a useful list of the groups that this person is in and all their ancestor groups.
-  # for most authorisation purposes, that's the set of groups of which this reader is considered a member.
-  # 
-  # Returns a scope.
-  #
-  def all_groups
-    Group.find_these(all_group_ids)
-  end
-  
-  def all_group_ids
-    self.groups.map(&:path_ids).flatten.uniq
-  end
-  
-  # Returns a list of the groups that this person is in along with their whole tree of super and subgroups.
-  # That's the list of groups that this person can see. It is larger than the list of groups that confer permission:
-  # this reader can see subgroups of his own groups in the directory, but he can't see their pages.
-  #
-  def all_visible_groups
-    Group.find_these(all_visible_group_ids)
-  end
-
-  def all_visible_group_ids
-    self.groups.map(&:tree_ids).flatten.uniq
-  end
-  
   def can_see? (this)
-    permitted_groups = this.permitted_groups
-    permitted_groups.empty? or in_any_of_these_groups?(permitted_groups)
+    this.groups.empty? or in_any_of_these_groups?(this.groups)
   end
     
   def in_any_of_these_groups? (grouplist)
-    (grouplist & all_groups).any?
+    (grouplist & groups).any?
   end
 
   def membership_of(group)

data/app/views/admin/groups/_group.html.haml

@@ -24,6 +24,4 @@
         = link_to_unless_current image('plus') + ' ' + t('reader_extension.add_subgroup'), new_admin_group_url(:parent_id => group.id), :class => 'action'
         = link_to_unless_current image('delta') + ' ' + t('reader_extension.edit_group'), edit_admin_group_url(group), :class => 'action'
         = link_to_unless_current image('minus') + ' ' + t('reader_extension.delete_group'), admin_group_url(group), :method => 'delete', :confirm => t("reader_extension.really_delete_group", :name => group.name), :class => 'action'
-
-= render :partial => 'group', :collection => group.children, :locals => {:level => level+1}
   

data/app/views/admin/messages/edit.html.haml

@@ -7,7 +7,7 @@
       = t('reader_extension.edit_message')
 
   - main.edit_form do
-    - form_for :message, @message, :url => admin_message_url(@message), :html => {:id => 'message_form', :method => 'put'} do |f|
+    - form_for :message, @message, :url => admin_message_path(@message), :html => {:id => 'message_form', :method => 'put'} do |f|
       = render :partial => 'form', :object => f
 
   - main.edit_popups do

data/app/views/admin/messages/new.html.haml

@@ -8,7 +8,7 @@
       = t('reader_extension.create_message')
 
   - main.edit_form do
-    - form_for :message, @message, :url => admin_messages_url, :html => {:id => 'message_form', :method => 'post'} do |f|
+    - form_for :message, @message, :url => admin_messages_path, :html => {:id => 'message_form', :method => 'post'} do |f|
       = render :partial => 'form', :object => f
 
   - main.edit_popups do

data/app/views/admin/messages/show.html.haml

@@ -48,7 +48,7 @@
           
     - delivery.options do
       - unless @message.administrative?
-        - form_for :message, @message, :url => deliver_admin_message_url(@message), :html => {"data-onsubmit_status"=>"Sending email messages…"} do |f|
+        - form_for :message, @message, :url => deliver_admin_message_path(@message), :html => {"data-onsubmit_status"=>"Sending email messages…"} do |f|
           %h3
             = t("reader_extension.send_to").titlecase + ":"
           .radio_group

data/app/views/admin/readers/_form.html.haml

@@ -5,7 +5,7 @@
   = render_region :form_top, :locals => {:f => f}
   
   - render_region :form, :locals => {:f => f} do |form|
-    - form.name do
+    - form.edit_name do
       %p
         = f.label :forename
         = f.text_field :forename, :class => "textbox activate", :size => 32, :maxlength => 100 
@@ -26,6 +26,30 @@
     - form.edit_password do
       = render "password_fields", :f => f
     
+    - form.edit_phone do
+      %p
+        = f.label :phone
+        = f.text_field :phone, :class => 'textbox'
+      %p
+        = f.label :mobile
+        = f.text_field :mobile, :class => 'textbox'
+
+    - form.edit_address do
+      %p
+        = f.label :post_line1, t('activerecord.attributes.reader.postal_address')
+        = f.text_field :post_line1, :class => 'textbox'
+        %br
+        = f.text_field :post_line2, :class => 'textbox'
+      %p
+        = f.label :post_city
+        = f.text_field :post_city, :class => 'textbox'
+      %p
+        = f.label :post_province
+        = f.text_field :post_province, :class => 'textbox'
+      %p
+        = f.label :postcode
+        = f.text_field :postcode, :class => 'textbox'
+
     - form.edit_description do
       - if Radiant.config['reader.show_descriptions?']
         %p

data/app/views/groups/_group.html.haml

@@ -10,7 +10,3 @@
   - if group.description
     %p
       = group.description
-      
-  - if with_children && group.children.any?
-    .indented
-      = render :partial => 'groups/group', :collection => group.children

data/app/views/groups/show.html.haml

@@ -3,9 +3,6 @@
   = t('reader_extension.separator')
   = link_to t('reader_extension.groups').titlecase, groups_url
   = t('reader_extension.separator')
-  - if @group.parent
-    = link_to @group.parent.name, group_url(@group.parent)
-    = t('reader_extension.separator')
 
 - content_for :title do
   = @group.name
@@ -20,18 +17,15 @@
 
 = render :partial => 'shared/standard_reader_parts'
 
-- content_for :main do
+- content_for :introduction do
   %p
     = t('reader_extension.download_group_as')
     = link_to t("reader_extension.csv_file"), group_url(@group, :format => :csv), :class => 'csv'
     = t('or')
-    = link_to t("reader_extension.vcard_file"), group_url(@group, :format => :vcard), :class => 'vcard'
-
-  - if @group.children.any?
-    %h2
-      = t('reader_extension.subgroups', :name => @group.name)
-    = render :partial => 'groups/group', :collection => @group.children
+    = link_to t("reader_extension.vcard_file"), group_url(@group, :format => :vcard), :class => 'vcards'
+    = t('reader_extension.or_individual_card')
 
+- content_for :main do
   - if @group.memberships.any?
     %h2
       = t('reader_extension.group_members')

data/app/views/memberships/_membership.html.haml

@@ -1,36 +1,4 @@
 - if membership
   - group = membership.group
   - reader = membership.reader
-  - if reader
-    %tr.reader
-      - if reader.unshareable?
-        %td{:colspan => 8}
-          = t("reader_extension.listing_denied")
-      - else
-        %td.gravatar
-          = link_to gravatar_for(reader, {:size => 18}, {:class => 'gravatar'}), reader_url(reader), :title => reader.preferred_name
-        %td.name
-          %h3
-            = link_to reader.preferred_name, reader_url(reader)
-        %td.role
-          %p
-            %span.role
-              = membership.role
-        %td.email
-          %p
-            = mail_to reader.email, nil, :encode => 'hex', :replace_at => " #{t('reader_extension.at')} "
-
-        - [:phone, :mobile].each do |field|
-          - value = reader.send(field)
-          - unless value.blank?
-            %td{:class => field.to_s}
-              %p
-                = value
-          - else
-            %td.missing
-              %p
-                = t("reader_extension.no_#{field}")
-
-        %td.vcard
-          %p
-            = link_to " ", reader_url(reader, :format => :vcard), :class => 'vcard'
+  = render :partial => 'readers/reader', :object => reader

data/app/views/readers/_directory.html.haml

@@ -1,27 +1,30 @@
 - case Radiant.config['reader.directory_visibility']
   - when 'public'
-    - if @groups.any?
-      %h2
-        = t('reader_extension.groups').titlecase
-      = render :partial => 'groups/group', :collection => @groups
+    .groups
+      - if @groups.any?
+        %h2
+          = t('reader_extension.groups').titlecase
+        = render :partial => 'groups/group', :collection => @groups
 
-    - if @readers.any?
-      %h2
-        = t('reader_extension.people').titlecase
-      = render :partial => 'readers/list', :locals => {:readers => @readers}
+    .readers
+      - if @readers.any?
+        %h2
+          = t('reader_extension.people').titlecase
+        = render :partial => 'readers/list', :locals => {:readers => @readers}
 
   - when 'grouped'
-    %h2
-      = t('reader_extension.your_groups')
-    - if @groups.any
-      = render :partial => 'groups/group', :collection => @groups
-    - else
-      = t('reader_extension.no_groups')
-
-    - if @readers.any?
+    .groups
       %h2
-        = t('reader_extension.your_group_members')
-      = render :partial => 'readers/list', :locals => {:readers => @readers}
+        = t('reader_extension.your_groups')
+      - if @groups.any
+        = render :partial => 'groups/group', :collection => @groups
+      - else
+        = t('reader_extension.no_groups')
+    .readers
+      - if @readers.any?
+        %h2
+          = t('reader_extension.your_group_members')
+        = render :partial => 'readers/list', :locals => {:readers => @readers}
 
   - else
     = t('reader_extension.no_directory')

data/app/views/readers/_reader.html.haml

@@ -7,10 +7,10 @@
       = link_to gravatar_for(reader, {:size => 18}, {:class => 'gravatar'}), reader_url(reader), :title => reader.preferred_name
     %td.name
       %h3
-        = link_to reader.preferred_name, reader_url(reader)
+        = link_to reader.name, reader_url(reader)
         - if reader.preferred_name != reader.name
           %span.note
-            = reader.name
+            = reader.preferred_name
     %td.email
       %p
         = mail_to reader.email, nil, :encode => 'hex', :replace_at => " #{t('reader_extension.at')} "

data/app/views/readers/show.html.haml

@@ -1,16 +1,15 @@
 - content_for :breadhead do
-  = link_to t('reader_extension.readers').titlecase, reader_index_url
+  = link_to t('reader_extension.readers_title').titlecase, reader_index_url
   = t('reader_extension.separator')
 
 - content_for :title do
-  = @reader.preferred_name
+  = @reader.name
 
 - content_for :introduction do
   - if Radiant.config['reader.show_descriptions?']
     = @reader.description
     %p
       = link_to t('reader_extension.edit_profile'), reader_edit_profile_url
-  
 
 - content_for :breadcrumbs do
   = yield :breadhead

data/config/locales/en.yml

@@ -153,7 +153,7 @@ en:
     create_message: "create message"
     create_new_message: "Send a new message"
     create_welcome: "create welcome"
-    csv_file: "CSV data"
+    csv_file: "csv data"
     dashboard:
       account: "Account settings"
       directory: "Directory"
@@ -185,7 +185,7 @@ en:
       vcard_link: "vcard"
     dont_fill: "Don't fill this in!"
     download_as: "Download as"
-    download_group_as: "Download the whole group as"
+    download_group_as: "You can download the whole group as"
     download_vcard: "Download vcard"
     edit_group: "edit group"
     edit_message: "Edit message"
@@ -280,6 +280,7 @@ en:
     message_preview_introduction: "This is a snapshot of the message that was delivered to you %{date}. If it contains instructions they may now be out of date."
     mistaken_access_error: "If you think that's a mistake, and that you should be allowed to do whatever it was you just tried to do, please contact your site administrator."
     must_be_empty: "must be empty"
+    name: "Name"
     name_and_login: "Name and Login"
     navigation:
       account: "Preferences"
@@ -319,6 +320,7 @@ en:
     optional_phone: "[phone]"
     or_edit_preferences: 'You can also <a href="%{url}">edit your account preferences</a>.'
     or_edit_profile: 'You can also <a href="%{url}">edit your profile</a> to add more information about yourself.'
+    or_individual_card: ' or click on an individual <img src="/images/furniture/vcard-16x16.png" /> to get their card.'
     other_messages: "Ad-hoc messages"
     page_access_explanation: "To restrict access to one or more groups, select them from the list below."
     page_not_public: "The page you have requested is not public. Please log in. If your account has the necessary permission you will be taken straight there."

data/lib/grouped_model.rb

@@ -54,7 +54,7 @@ module GroupedModel
       named_scope :visible_to, lambda { |reader| 
         conditions = "pp.group_id IS NULL"
         if reader && reader.is_grouped?
-          ids = reader.all_group_ids
+          ids = reader.group_ids
           conditions = ["#{conditions} OR pp.group_id IS NULL OR pp.group_id IN(#{ids.map{"?"}.join(',')})", *ids]
         end
         {
@@ -94,11 +94,10 @@ module GroupedModel
       end
       
       named_scope :belonging_to, lambda { |group|
-        group_ids = group.subtree_ids
         {
           :joins => "INNER JOIN permissions as pp on pp.permitted_id = #{self.table_name}.id AND pp.permitted_type = '#{self.to_s}'", 
           :group => column_names.map { |n| self.table_name + '.' + n }.join(','),
-          :conditions => ["pp.group_id IN (#{group_ids.map{"?"}.join(',')})", *group_ids],
+          :conditions => ["pp.group_id = ?", group.id],
           :readonly => false
         }
       }
@@ -114,49 +113,31 @@ module GroupedModel
 
   module GroupedInstanceMethods
 
-    # The list of groups that is allowed to see this object. This will include the groups directly
-    # associated and their descendant subgroups.
-    #
-    def permitted_groups
-      if permitted_group_ids.any?
-        Group.find(permitted_group_ids)
-      else
-        []
-      end
-    end
-
-    def permitted_group_ids
-      # in GroupedPage this is chained to include groups inherited from ancestor pages
-      # while here the subtrees provide inheritance from ancestor groups.
-      #
-      groups.map(&:subtree_ids).flatten.uniq
-    end
-
     def visible_to?(reader)
-      return true if self.permitted_groups.empty?
+      return true if self.groups.empty?
       return false if reader.nil?
       return true if reader.is_admin?
-      return (reader.groups & self.permitted_groups).any?
+      return (reader.groups & self.groups).any?
     end
 
     def group
-      if self.permitted_groups.length == 1
-        self.permitted_groups.first
+      if self.groups.length == 1
+        self.groups.first
       else
         nil
       end
     end
     
     def visible?
-      permitted_groups.empty?
+      groups.empty?
     end
 
     def permitted_readers
-      permitted_groups.any? ? Reader.in_groups(permitted_groups) : Reader.scoped({})
+      groups.any? ? Reader.in_groups(groups) : Reader.scoped({})
     end
     
     def has_group?(group)
-      return self.permitted_groups.include?(group)
+      return self.groups.include?(group)
     end
     
     def permit(group)

data/lib/grouped_page.rb

@@ -5,7 +5,7 @@ module GroupedPage
       has_groups
       has_one :homegroup, :foreign_key => 'homepage_id', :class_name => 'Group'
       include InstanceMethods
-      alias_method_chain :permitted_group_ids, :inheritance
+      alias_method_chain :group_ids, :inheritance
       alias_method_chain :cache?, :restrictions
     }
   end
@@ -23,15 +23,15 @@ module GroupedPage
       self.ancestors.map(&:group_ids).flatten.uniq
     end
 
-    def permitted_group_ids_with_inheritance
-      (permitted_group_ids_without_inheritance + inherited_group_ids).flatten.uniq
+    def group_ids_with_inheritance
+      (group_ids_without_inheritance + inherited_group_ids).flatten.uniq
     end
 
     # this is regrettably expensive and I plan to replace it with a 
-    # private? method that would be cascaded on page update
+    # private? setter that would be cascaded on page update
     #
     def restricted?
-      self.permitted_groups.any?
+      self.groups.any?
     end
     
     def cache_with_restrictions?

data/lib/radiant-reader-extension.rb

@@ -1,5 +1,5 @@
 module RadiantReaderExtension
-  VERSION = '3.0.32'
+  VERSION = '3.0.33'
   SUMMARY = %q{Reader/viewer/visitor registration, login and access-control for Radiant CMS}
   DESCRIPTION = %q{Provides reader/member/user registration and management functions including password-reminder, group-based page access control and administrative email.}
   URL = "http://radiant.spanner.org/reader"

data/lib/reader_admin_ui.rb

@@ -29,7 +29,7 @@ module ReaderAdminUI
         OpenStruct.new.tap do |reader|
           reader.edit = Radiant::AdminUI::RegionSet.new do |edit|
             edit.main.concat %w{edit_header edit_form}
-            edit.form.concat %w{edit_name edit_email edit_nickname edit_password reader_groups edit_description edit_notes}
+            edit.form.concat %w{edit_name edit_email edit_nickname edit_password reader_groups edit_address edit_phone edit_description edit_notes}
             edit.form_bottom.concat %w{edit_timestamp edit_buttons}
           end
           reader.index = Radiant::AdminUI::RegionSet.new do |index|

data/lib/reader_tags.rb

@@ -36,6 +36,8 @@ module ReaderTags
   end
   tag 'readers:each' do |tag|
     tag.locals.readers = get_readers(tag)
+    
+    Rails.logger.warn "readers:each: tlr has #{tag.locals.readers.size} readers"
     tag.render('reader_list', tag.attr.dup, &tag.block)
   end
 
@@ -73,7 +75,7 @@ module ReaderTags
     we are on an uncached page.
   }
   tag 'reader' do |tag|
-    tag.expand if tag.locals.reader = get_reader(tag)
+    tag.expand if tag.locals.reader ||= get_reader(tag)
   end
 
   [:name, :forename, :surname, :nickname, :preferred_name, :preferred_informal_name, :email, :description, :login, :honorific].each do |field|

data/spec/datasets/readers_dataset.rb

@@ -86,8 +86,7 @@ From <r:sender:name />
       create_model :group, symbol, { 
         :name => name,
         :slug => name.downcase,
-        :description => "#{name} group",
-        :parent_id => @group_id
+        :description => "#{name} group"
       }.merge(attributes)
       if block_given?
         @group_id = group_id(symbol)

data/spec/models/group_spec.rb

@@ -49,45 +49,7 @@ describe Group do
       g.slug.should == 'testy-group'
     end
   end
-  
-  describe "in a hierarchy" do
-    it "should have parent and children relations" do
-      group = groups(:subgroup)
-      group.respond_to?(:parent).should be_true
-      group.respond_to?(:children).should be_true
-      group.children.should =~ [groups(:subsubgroup), groups(:anothersubsubgroup)]
-      group.parent.should == groups(:supergroup)
-    end
-    
-    it "should have descendants and ancestors" do
-      groups(:subsubgroup).path.should == [groups(:supergroup), groups(:subgroup), groups(:subsubgroup)]
-      groups(:subsubgroup).root.should == groups(:supergroup)
-      groups(:supergroup).subtree.should =~ [groups(:supergroup), groups(:subgroup), groups(:subsubgroup), groups(:anothersubsubgroup)]
-    end
-
-    it "should have a root group" do
-      [:supergroup, :subgroup, :subsubgroup].each do |g|
-        groups(g).root.should == groups(:supergroup)
-      end
-    end
-
-    it "should inherit memberships from descendants" do
-      groups(:supergroup).members.should =~ [readers(:normal), readers(:another)]
-    end
-    
-    it "should not inherit memberships from ancestors" do
-      groups(:subsubgroup).members.should be_empty
-    end
     
-    it "should inherit permissions from ancestors" do
-      groups(:subsubgroup).pages.should =~ [pages(:child), pages(:child_2)]
-    end
-
-    it "should not inherit permissions from descendants" do
-      groups(:supergroup).pages.should be_empty
-    end
-  end
-  
   describe "directory visibility" do
     describe "when directory is grouped" do
       before do
@@ -98,18 +60,6 @@ describe Group do
         groups(:subgroup).visible_to?(readers(:normal)).should be_true
       end
       
-      it "should be visible to members of ancestor groups" do
-        groups(:supergroup).visible_to?(readers(:normal)).should be_true
-      end
-      
-      it "should be visible to members of descendant groups" do
-        groups(:subsubgroup).visible_to?(readers(:normal)).should be_true
-      end
-      
-      it "should not be visible to members of groups outside the family" do
-        groups(:subgroup).visible_to?(readers(:inactive)).should be_false
-      end
-      
       it "should not be visible to readers without groups" do
         groups(:subgroup).visible_to?(readers(:ungrouped)).should be_false
       end

data/spec/models/reader_page_spec.rb

@@ -114,7 +114,7 @@ describe ReaderPage do
       it "should allow access only to groups to which the reader belongs" do
         lambda { Page.find_by_path("/directory/normal") }.should_not raise_error
         lambda { Page.find_by_path("/directory/special") }.should raise_error(ReaderError::AccessDenied)
-        Page.find_by_path("/directory").groups.should =~ readers(:normal).all_visible_groups
+        Page.find_by_path("/directory").groups.should =~ readers(:normal).groups
       end
 
     end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: radiant-reader-extension
 version: !ruby/object:Gem::Version 
-  hash: 71
+  hash: 69
   prerelease: 
   segments: 
   - 3
   - 0
-  - 32
-  version: 3.0.32
+  - 33
+  version: 3.0.33
 platform: ruby
 authors: 
 - William Ross
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-11-14 00:00:00 +00:00
+date: 2011-11-21 00:00:00 +00:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -325,14 +325,17 @@ files:
 - public/images/furniture/csv.png
 - public/images/furniture/csv_tiny.png
 - public/images/furniture/no_gravatar.png
-- public/images/furniture/vcard.png
+- public/images/furniture/vcard-16x16.png
+- public/images/furniture/vcard-24x24.png
+- public/images/furniture/vcard-32x32.png
+- public/images/furniture/vcard-48x48.png
 - public/images/furniture/vcard_tiny.png
+- public/images/furniture/vcards.png
 - public/javascripts/admin/reader.js
 - public/javascripts/reader.js
 - public/stylesheets/sass/admin/reader.sass
 - public/stylesheets/sass/admin/reader_group.sass
 - public/stylesheets/sass/reader.sass
-- radiant-reader-extension-3.0.31.gem
 - radiant-reader-extension.gemspec
 - Rakefile
 - reader_extension.rb
@@ -362,7 +365,7 @@ has_rdoc: true
 homepage: http://radiant.spanner.org/reader
 licenses: []
 
-post_install_message: "\n  Add this to your radiant project with a line in your Gemfile:\n\n    gem 'radiant-reader-extension', '~> 3.0.32'\n\n  and if you haven't already, remember to enable ActionMailer in your\n  project's config/environment.rb.\n  "
+post_install_message: "\n  Add this to your radiant project with a line in your Gemfile:\n\n    gem 'radiant-reader-extension', '~> 3.0.33'\n\n  and if you haven't already, remember to enable ActionMailer in your\n  project's config/environment.rb.\n  "
 rdoc_options: []
 
 require_paths: