]> source.dussan.org Git - redmine.git/commitdiff
Activity enhancements:
authorJean-Philippe Lang <jp_lang@yahoo.fr>
Tue, 11 Mar 2008 19:33:38 +0000 (19:33 +0000)
committerJean-Philippe Lang <jp_lang@yahoo.fr>
Tue, 11 Mar 2008 19:33:38 +0000 (19:33 +0000)
* overall activity view and feed added, link is available on the project list (#423, #494)
* switch added on the project activity view to include subprojects (closes #530)

git-svn-id: http://redmine.rubyforge.org/svn/trunk@1227 e93f8b46-1217-0410-a6f0-8f06a7374b81

39 files changed:
app/controllers/projects_controller.rb
app/models/attachment.rb
app/models/changeset.rb
app/models/project.rb
app/models/wiki_content.rb
app/views/common/feed.atom.rxml
app/views/projects/activity.rhtml
app/views/projects/list.rhtml
app/views/welcome/index.rhtml
lang/bg.yml
lang/cs.yml
lang/da.yml
lang/de.yml
lang/en.yml
lang/es.yml
lang/fi.yml
lang/fr.yml
lang/he.yml
lang/it.yml
lang/ja.yml
lang/ko.yml
lang/lt.yml
lang/nl.yml
lang/pl.yml
lang/pt-br.yml
lang/pt.yml
lang/ro.yml
lang/ru.yml
lang/sr.yml
lang/sv.yml
lang/uk.yml
lang/zh-tw.yml
lang/zh.yml
lib/ar_condition.rb
public/stylesheets/application.css
test/fixtures/issues.yml
test/fixtures/messages.yml
test/functional/projects_controller_test.rb
test/unit/project_test.rb

index 7e4cc81f3fb592dc53c23b4a20b107142967bfe2..9268a19ea4fa56de94071c09b56ae660f58cf5be 100644 (file)
@@ -24,8 +24,9 @@ class ProjectsController < ApplicationController
   menu_item :settings, :only => :settings
   menu_item :issues, :only => [:changelog]
   
-  before_filter :find_project, :except => [ :index, :list, :add ]
-  before_filter :authorize, :except => [ :index, :list, :add, :archive, :unarchive, :destroy ]
+  before_filter :find_project, :except => [ :index, :list, :add, :activity ]
+  before_filter :find_optional_project, :only => :activity
+  before_filter :authorize, :except => [ :index, :list, :add, :archive, :unarchive, :destroy, :activity ]
   before_filter :require_admin, :only => [ :add, :archive, :unarchive, :destroy ]
   accept_key_auth :activity, :calendar
   
@@ -228,12 +229,14 @@ class ProjectsController < ApplicationController
     @date_from = @date_to - @days
     
     @event_types = %w(issues news files documents changesets wiki_pages messages)
-    @event_types.delete('wiki_pages') unless @project.wiki
-    @event_types.delete('changesets') unless @project.repository
-    @event_types.delete('messages') unless @project.boards.any?
-    # only show what the user is allowed to view
-    @event_types = @event_types.select {|o| User.current.allowed_to?("view_#{o}".to_sym, @project)}
-    
+    if @project
+      @event_types.delete('wiki_pages') unless @project.wiki
+      @event_types.delete('changesets') unless @project.repository
+      @event_types.delete('messages') unless @project.boards.any?
+      # only show what the user is allowed to view
+      @event_types = @event_types.select {|o| User.current.allowed_to?("view_#{o}".to_sym, @project)}
+      @with_subprojects = params[:with_subprojects].nil? ? Setting.display_subprojects_issues? : (params[:with_subprojects] == '1')
+    end
     @scope = @event_types.select {|t| params["show_#{t}"]}
     # default events if none is specified in parameters
     @scope = (@event_types - %w(wiki_pages messages))if @scope.empty?
@@ -241,21 +244,41 @@ class ProjectsController < ApplicationController
     @events = []    
     
     if @scope.include?('issues')
-      @events += @project.issues.find(:all, :include => [:author, :tracker], :conditions => ["#{Issue.table_name}.created_on>=? and #{Issue.table_name}.created_on<=?", @date_from, @date_to] )
-      @events += @project.issues_status_changes(@date_from, @date_to)
+      cond = ARCondition.new(Project.allowed_to_condition(User.current, :view_issues, :project => @project, :with_subprojects => @with_subprojects))
+      cond.add(["#{Issue.table_name}.created_on BETWEEN ? AND ?", @date_from, @date_to])
+      @events += Issue.find(:all, :include => [:project, :author, :tracker], :conditions => cond.conditions)
+      
+      cond = ARCondition.new(Project.allowed_to_condition(User.current, :view_issues, :project => @project, :with_subprojects => @with_subprojects))
+      cond.add(["#{Journal.table_name}.journalized_type = 'Issue' AND #{JournalDetail.table_name}.prop_key = 'status_id' AND #{Journal.table_name}.created_on BETWEEN ? AND ?", @date_from, @date_to])
+      @events += Journal.find(:all, :include => [{:issue => :project}, :details, :user], :conditions => cond.conditions)
     end
     
     if @scope.include?('news')
-      @events += @project.news.find(:all, :conditions => ["#{News.table_name}.created_on>=? and #{News.table_name}.created_on<=?", @date_from, @date_to], :include => :author )
+      cond = ARCondition.new(Project.allowed_to_condition(User.current, :view_news, :project => @project, :with_subprojects => @with_subprojects))
+      cond.add(["#{News.table_name}.created_on BETWEEN ? AND ?", @date_from, @date_to])
+      @events += News.find(:all, :include => [:project, :author], :conditions => cond.conditions)
     end
     
     if @scope.include?('files')
-      @events += Attachment.find(:all, :select => "#{Attachment.table_name}.*", :joins => "LEFT JOIN #{Version.table_name} ON #{Version.table_name}.id = #{Attachment.table_name}.container_id", :conditions => ["#{Attachment.table_name}.container_type='Version' and #{Version.table_name}.project_id=? and #{Attachment.table_name}.created_on>=? and #{Attachment.table_name}.created_on<=?", @project.id, @date_from, @date_to], :include => :author )
+      cond = ARCondition.new(Project.allowed_to_condition(User.current, :view_files, :project => @project, :with_subprojects => @with_subprojects))
+      cond.add(["#{Attachment.table_name}.created_on BETWEEN ? AND ?", @date_from, @date_to])
+      @events += Attachment.find(:all, :select => "#{Attachment.table_name}.*", 
+                                       :joins => "LEFT JOIN #{Version.table_name} ON #{Attachment.table_name}.container_type='Version' AND #{Version.table_name}.id = #{Attachment.table_name}.container_id " +
+                                                 "LEFT JOIN #{Project.table_name} ON #{Version.table_name}.project_id = #{Project.table_name}.id",
+                                       :conditions => cond.conditions)
     end
     
     if @scope.include?('documents')
-      @events += @project.documents.find(:all, :conditions => ["#{Document.table_name}.created_on>=? and #{Document.table_name}.created_on<=?", @date_from, @date_to] )
-      @events += Attachment.find(:all, :select => "attachments.*", :joins => "LEFT JOIN #{Document.table_name} ON #{Document.table_name}.id = #{Attachment.table_name}.container_id", :conditions => ["#{Attachment.table_name}.container_type='Document' and #{Document.table_name}.project_id=? and #{Attachment.table_name}.created_on>=? and #{Attachment.table_name}.created_on<=?", @project.id, @date_from, @date_to], :include => :author )
+      cond = ARCondition.new(Project.allowed_to_condition(User.current, :view_documents, :project => @project, :with_subprojects => @with_subprojects))
+      cond.add(["#{Document.table_name}.created_on BETWEEN ? AND ?", @date_from, @date_to])
+      @events += Document.find(:all, :include => :project, :conditions => cond.conditions)
+      
+      cond = ARCondition.new(Project.allowed_to_condition(User.current, :view_documents, :project => @project, :with_subprojects => @with_subprojects))
+      cond.add(["#{Attachment.table_name}.created_on BETWEEN ? AND ?", @date_from, @date_to])
+      @events += Attachment.find(:all, :select => "#{Attachment.table_name}.*", 
+                                       :joins => "LEFT JOIN #{Document.table_name} ON #{Attachment.table_name}.container_type='Document' AND #{Document.table_name}.id = #{Attachment.table_name}.container_id " +
+                                                 "LEFT JOIN #{Project.table_name} ON #{Document.table_name}.project_id = #{Project.table_name}.id",
+                                       :conditions => cond.conditions)
     end
     
     if @scope.include?('wiki_pages')
@@ -264,28 +287,31 @@ class ProjectsController < ApplicationController
                "#{WikiContent.versioned_table_name}.page_id, #{WikiContent.versioned_table_name}.author_id, " +
                "#{WikiContent.versioned_table_name}.id"
       joins = "LEFT JOIN #{WikiPage.table_name} ON #{WikiPage.table_name}.id = #{WikiContent.versioned_table_name}.page_id " +
-              "LEFT JOIN #{Wiki.table_name} ON #{Wiki.table_name}.id = #{WikiPage.table_name}.wiki_id "
-      conditions = ["#{Wiki.table_name}.project_id = ? AND #{WikiContent.versioned_table_name}.updated_on BETWEEN ? AND ?",
-                    @project.id, @date_from, @date_to]
+              "LEFT JOIN #{Wiki.table_name} ON #{Wiki.table_name}.id = #{WikiPage.table_name}.wiki_id " +
+              "LEFT JOIN #{Project.table_name} ON #{Project.table_name}.id = #{Wiki.table_name}.project_id"
 
-      @events += WikiContent.versioned_class.find(:all, :select => select, :joins => joins, :conditions => conditions)
+      cond = ARCondition.new(Project.allowed_to_condition(User.current, :view_wiki_pages, :project => @project, :with_subprojects => @with_subprojects))
+      cond.add(["#{WikiContent.versioned_table_name}.updated_on BETWEEN ? AND ?", @date_from, @date_to])
+      @events += WikiContent.versioned_class.find(:all, :select => select, :joins => joins, :conditions => cond.conditions)
     end
 
     if @scope.include?('changesets')
-      @events += Changeset.find(:all, :include => :repository, :conditions => ["#{Repository.table_name}.project_id = ? AND #{Changeset.table_name}.committed_on BETWEEN ? AND ?", @project.id, @date_from, @date_to])
+      cond = ARCondition.new(Project.allowed_to_condition(User.current, :view_changesets, :project => @project, :with_subprojects => @with_subprojects))
+      cond.add(["#{Changeset.table_name}.committed_on BETWEEN ? AND ?", @date_from, @date_to])
+      @events += Changeset.find(:all, :include => {:repository => :project}, :conditions => cond.conditions)
     end
     
     if @scope.include?('messages')
-      @events += Message.find(:all, 
-                              :include => [:board, :author], 
-                              :conditions => ["#{Board.table_name}.project_id=? AND #{Message.table_name}.created_on BETWEEN ? AND ?", @project.id, @date_from, @date_to])
+      cond = ARCondition.new(Project.allowed_to_condition(User.current, :view_messages, :project => @project, :with_subprojects => @with_subprojects))
+      cond.add(["#{Message.table_name}.created_on BETWEEN ? AND ?", @date_from, @date_to])
+      @events += Message.find(:all, :include => [{:board => :project}, :author], :conditions => cond.conditions)
     end
     
     @events_by_day = @events.group_by(&:event_date)
     
     respond_to do |format|
       format.html { render :layout => false if request.xhr? }
-      format.atom { render_feed(@events, :title => "#{@project.name}: #{l(:label_activity)}") }
+      format.atom { render_feed(@events, :title => "#{@project || Setting.app_title}: #{l(:label_activity)}") }
     end
   end
   
@@ -381,6 +407,14 @@ private
     render_404
   end
   
+  def find_optional_project
+    return true unless params[:id]
+    @project = Project.find(params[:id])
+    authorize
+  rescue ActiveRecord::RecordNotFound
+    render_404
+  end
+
   def retrieve_selected_tracker_ids(selectable_trackers)
     if ids = params[:tracker_ids]
       @selected_tracker_ids = (ids.is_a? Array) ? ids.collect { |id| id.to_i.to_s } : ids.split('/').collect { |id| id.to_i.to_s }
index c9783b9ce5d53d4f049358419f2da1f2043523fb..cdcb8d231dce82f92a5090bfeacc43d702a64dea 100644 (file)
@@ -26,7 +26,6 @@ class Attachment < ActiveRecord::Base
   validates_length_of :disk_filename, :maximum => 255
 
   acts_as_event :title => :filename,
-                :description => :filename,
                 :url => Proc.new {|o| {:controller => 'attachments', :action => 'download', :id => o.id}}
 
   cattr_accessor :storage_path
index 6bd15b1581bc133658be66d00f56cbf8397dd4a1..dbe06935dc8126d4533407001e4e903a0b7007e8 100644 (file)
@@ -45,6 +45,10 @@ class Changeset < ActiveRecord::Base
     super
   end
   
+  def project
+    repository.project
+  end
+  
   def after_create
     scan_comment_for_issue_ids
   end
index 95a201454c7344a49d7cf7bca381eabf82c3b065..ad2f1fb814be2bf1880901a7f6f3a00051ebfa98 100644 (file)
@@ -84,16 +84,6 @@ class Project < ActiveRecord::Base
     end 
   end
 
-  # Return all issues status changes for the project between the 2 given dates
-  def issues_status_changes(from, to)
-    Journal.find(:all, :include => [:issue, :details, :user],
-                       :conditions => ["#{Journal.table_name}.journalized_type = 'Issue'" +
-                                       " AND #{Issue.table_name}.project_id = ?" +
-                                       " AND #{JournalDetail.table_name}.prop_key = 'status_id'" +
-                                       " AND #{Journal.table_name}.created_on BETWEEN ? AND ?",
-                                       id, from, to+1])
-  end
-
   # returns latest created projects
   # non public projects will be returned only if user is a member of those
   def self.latest(user=nil, count=5)
@@ -110,19 +100,28 @@ class Project < ActiveRecord::Base
     end
   end
   
-  def self.allowed_to_condition(user, permission)
+  def self.allowed_to_condition(user, permission, options={})
     statements = []
-    active_statement = "#{Project.table_name}.status=#{Project::STATUS_ACTIVE}"
+    base_statement = "#{Project.table_name}.status=#{Project::STATUS_ACTIVE}"
+    if options[:project]
+      project_statement = "#{Project.table_name}.id = #{options[:project].id}"
+      project_statement << " OR #{Project.table_name}.parent_id = #{options[:project].id}" if options[:with_subprojects]
+      base_statement = "(#{project_statement}) AND (#{base_statement})"
+    end
     if user.admin?
       # no restriction
     elsif user.logged?
       statements << "#{Project.table_name}.is_public = #{connection.quoted_true}" if Role.non_member.allowed_to?(permission)
       allowed_project_ids = user.memberships.select {|m| m.role.allowed_to?(permission)}.collect {|m| m.project_id}
       statements << "#{Project.table_name}.id IN (#{allowed_project_ids.join(',')})" if allowed_project_ids.any?
+    elsif Role.anonymous.allowed_to?(permission)
+      # anonymous user allowed on public project
+      statements << "#{Project.table_name}.is_public = #{connection.quoted_true}" 
     else
-      statements << "#{Project.table_name}.is_public = #{connection.quoted_true}" if Role.anonymous.allowed_to?(permission)
+      # anonymous user is not authorized
+      statements << "1=0"
     end
-    statements.empty? ? active_statement : "(#{active_statement} AND (#{statements.join(' OR ')}))"
+    statements.empty? ? base_statement : "((#{base_statement}) AND (#{statements.join(' OR ')}))"
   end
   
   def self.find(*args)
index c307beb1ddd695bc1390bacbc7254b54a4393bc9..13915c2740af42b596f178e689ff905d78c3d49f 100644 (file)
@@ -61,6 +61,10 @@ class WikiContent < ActiveRecord::Base
       end      
     end
     
+    def project
+      page.project
+    end
+    
     # Returns the previous version or nil
     def previous
       @previous ||= WikiContent::Version.find(:first, 
index 6695565f4bf57fee2f0aca9cc94880e90bd7cf85..b5cbeeed9c743d72c02924524df86efbce66dc1e 100644 (file)
@@ -9,9 +9,10 @@ xml.feed "xmlns" => "http://www.w3.org/2005/Atom" do
   xml.generator(:uri => Redmine::Info.url, :version => Redmine::VERSION) { xml.text! Redmine::Info.versioned_name; }
   @items.each do |item|
     xml.entry do
+      url = url_for(item.event_url(:only_path => false))
       xml.title truncate(item.event_title, 100)
-      xml.link "rel" => "alternate", "href" => url_for(item.event_url(:only_path => false))
-      xml.id url_for(item.event_url(:only_path => false))
+      xml.link "rel" => "alternate", "href" => url
+      xml.id url
       xml.updated item.event_datetime.xmlschema
       author = item.event_author if item.respond_to?(:author)
       xml.author do
index 95c8d485fa3b8d61601a76ac2b3f6f0ce0583c51..e8832fccc806877211d4831fad837c445d44b3a4 100644 (file)
@@ -6,7 +6,7 @@
 <dl>
 <% @events_by_day[day].sort {|x,y| y.event_datetime <=> x.event_datetime }.each do |e| -%>
   <dt class="<%= e.class.name.downcase %>"><span class="time"><%= format_time(e.event_datetime, false) %></span>
-  <%= link_to h(truncate(e.event_title, 100)), e.event_url %></dt>
+  <%= content_tag('span', h(e.project), :class => 'project') if @project.nil? || @project != e.project %> <%= link_to h(truncate(e.event_title, 100)), e.event_url %></dt>
   <dd><% unless e.event_description.blank? -%>
   <span class="description"><%= format_activity_description(e.event_description) %></span><br />
   <% end %>
 </p>
 
 <% content_for :header_tags do %>
-<%= auto_discovery_link_tag(:atom, params.merge(:format => 'atom', :year => nil, :month => nil, :key => User.current.rss_key).delete_if{|k,v|k=="commit"}) %>
+<%= auto_discovery_link_tag(:atom, params.merge(:format => 'atom', :year => nil, :month => nil, :key => User.current.rss_key)) %>
 <% end %>
 
 <% content_for :sidebar do %>
-<% form_tag do %>
+<% form_tag({}, :method => :get) do %>
 <h3><%= l(:label_activity) %></h3>
 <p><% @event_types.each do |t| %>
 <label><%= check_box_tag "show_#{t}", 1, @scope.include?(t) %> <%= l("label_#{t.singularize}_plural")%></label><br />
 <% end %></p>
-<p><%= submit_tag l(:button_apply), :class => 'button-small' %></p>
+<% if @project && @project.active_children.any? %>
+    <p><label><%= check_box_tag 'with_subprojects', 1, @with_subprojects %> <%=l(:label_subproject_plural)%></label></p>
+    <%= hidden_field_tag 'with_subprojects', 0 %>
+<% end %>
+<p><%= submit_tag l(:button_apply), :class => 'button-small', :name => nil %></p>
 <% end %>
 <% end %>
 
index 15ea06483403f27a17c27e5a57fd1cce0188dd22..b8bb62ebb05ac8f3fa24c86dbc6cc58a11dc2919 100644 (file)
@@ -1,3 +1,8 @@
+<div class="contextual">
+    <%= link_to l(:label_issue_view_all), { :controller => 'issues' } %> |
+    <%= link_to l(:label_overall_activity), { :controller => 'projects', :action => 'activity' }%>
+</div>
+
 <h2><%=l(:label_project_plural)%></h2>
 
 <% @project_tree.keys.sort.each do |project| %>
index d618fa6e1f844109de5e4973207305f3089bc06b..5da5a1ed361ffa9b010e43217ddb8d17f2ac7a20 100644 (file)
@@ -26,5 +26,8 @@
 </div> 
 
 <% content_for :header_tags do %>
-<%= auto_discovery_link_tag(:atom, {:controller => 'news', :action => 'index', :key => User.current.rss_key, :format => 'atom'}, {:title => l(:label_news_latest)}) %>
+<%= auto_discovery_link_tag(:atom, {:controller => 'news', :action => 'index', :key => User.current.rss_key, :format => 'atom'},
+                                   :title => "#{Setting.app_title}: #{l(:label_news_latest)}") %>
+<%= auto_discovery_link_tag(:atom, {:controller => 'projects', :action => 'activity', :key => User.current.rss_key, :format => 'atom'},
+                                   :title => "#{Setting.app_title}: #{l(:label_activity)}") %>
 <% end %>
index 87808230e9f58a7a14e488b89768c5e09a3e3166..99c8eb29655eb841e3b85e1eacccf80f26853360 100644 (file)
@@ -613,3 +613,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index c5cef37775ec4abee219c6e57e95a4b29f4aef71..7d11bb6a8b9faea73d4c5884c32485f3cc4f487a 100644 (file)
@@ -613,3 +613,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index ed2eb26a7b487e5c73c6c748552f84b27e36982c..825f15ecc0463617fa0fbe0f964e92b1d7e70283 100644 (file)
@@ -615,3 +615,4 @@ field_comments_sorting: Display comments
 text_reassign_time_entries: 'Reassign reported hours to this issue:'
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
+label_overall_activity: Overall activity
index aa493853431e85f7b607731ecb09bdfb40896190..eeb851018cb340e2f283d24663743e232ce08d25 100644 (file)
@@ -614,3 +614,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index af6c8faa9f716c42c2bf81d42fbbbff5638a51a8..26c7b476636569d3b19562c7516093593d4d1a06 100644 (file)
@@ -280,6 +280,7 @@ label_last_updates: Last updated
 label_last_updates_plural: %d last updated
 label_registered_on: Registered on
 label_activity: Activity
+label_overall_activity: Overall activity
 label_new: New
 label_logged_as: Logged as
 label_environment: Environment
index 47ee41524f8c80e8cdd0a75222d4cf7871277d06..5540201551216d0f74a1cbe8b38185f9d9a0e2e7 100644 (file)
@@ -616,3 +616,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index b84d6cd78a6ea66df9ca23640079f3f26a1b0f18..4923440fdec50840a087cc9386b1da9d720b9b43 100644 (file)
@@ -620,3 +620,4 @@ setting_display_subprojects_issues: Display subprojects issues on main projects
 field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
+label_overall_activity: Overall activity
index 243088f049798873497ce763016089a9e6b07568..c7cfc54e5d39cb193796674477a4936f87f88351 100644 (file)
@@ -165,6 +165,7 @@ field_start_page: Page de démarrage
 field_subproject: Sous-projet
 field_hours: Heures
 field_activity: Activité
+label_overall_activity: Activité globale
 field_spent_on: Date
 field_identifier: Identifiant
 field_is_filter: Utilisé comme filtre
@@ -452,7 +453,7 @@ label_start_to_start: début à début
 label_start_to_end: début à fin
 label_stay_logged_in: Rester connecté
 label_disabled: désactivé
-label_show_completed_versions: Voire les versions passées
+label_show_completed_versions: Voir les versions passées
 label_me: moi
 label_board: Forum
 label_board_new: Nouveau forum
index 972b1134adef1e9f1c331987ed3e1cac5b6d2c8a..5d585eb610484a136e7f774880dc0cfb57df9778 100644 (file)
@@ -613,3 +613,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index c3250ebbc32eabc8a1252be2f8d2ede81c2a62b1..1d2cb742d72e11f5e238f8abfcf62364d9534085 100644 (file)
@@ -613,3 +613,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index 79bb572a834e61c632ba2a674f0bc34bd73e9d89..de883fe582104cf100b64e894234f6404c11f515 100644 (file)
@@ -614,3 +614,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index 3ca07763155974b762d59238df6c13ebe8b22004..bfe7be912e9766dff1abe07d999abdaf07e53581 100644 (file)
@@ -613,3 +613,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index 93f48580f79ca2047b0b401b3865e4c4854890bc..b5bb15df08166a13f8f777aee037e7a3471d88fc 100644 (file)
@@ -614,3 +614,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index e906658788fd28e3f13688a3123ce335e75f323c..65d13ddab9525265b2e8527521ccc3a055bdf62a 100644 (file)
@@ -614,3 +614,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index 7003fb8714574039541997260d6f840fbb806a24..10a8885d1006bd81085c174fb0103a9c2b77a693 100644 (file)
@@ -613,3 +613,4 @@ setting_display_subprojects_issues: Display subprojects issues on main projects
 field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
+label_overall_activity: Overall activity
index 10836d336df08d380de7a44f23ed04d5dd303620..3bd38a55765265b5b0279b9c904df8cf2a7ea094 100644 (file)
@@ -613,3 +613,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order\r
 label_preferences: Preferences\r
 setting_display_subprojects_issues: Display subprojects issues on main projects by default\r
+label_overall_activity: Overall activity\r
index f59e3b06ad446ce9b0354360d0f9059734753427..65d26b1e367064aec08c09f680940ece182da8ba 100644 (file)
@@ -613,3 +613,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index cb7bb69be8ea722ad36cfd022ad62d7994251e12..8e91d277fd0f71dbec1bf54037d12727d020d490 100644 (file)
@@ -613,3 +613,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index f12b2264f219a3e8cee2e975c84050ebafe89b44..f08271e4585391a587292a8b48275e35d9b7cce9 100644 (file)
@@ -617,3 +617,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index e59b3cba92d43eb1006201f56bbb5e892b472cb6..82d06f93b79c1b5edba25aaabfc6ea48971c46f1 100644 (file)
@@ -614,3 +614,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index 5ff3f8f17f83f11447ed600c3411b9cbf95ff0c9..cc9fcd212367ded295bd75643b5dcbebf49a3846 100644 (file)
@@ -614,3 +614,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index 8c33c7806e4e9dfa9cdccb86987fe0c688964959..08a36e3527351db08f3e27c5f4cbe76e3445b929 100644 (file)
@@ -615,3 +615,4 @@ field_comments_sorting: Display comments
 label_reverse_chronological_order: In reverse chronological order
 label_preferences: Preferences
 setting_display_subprojects_issues: Display subprojects issues on main projects by default
+label_overall_activity: Overall activity
index 9334c72bb3d8f024f270849fe5ac719cc2149511..95e5afbb428eac9e10ca33529aa4cef710fd65a5 100644 (file)
@@ -614,3 +614,4 @@ default_activity_development: 開發
 enumeration_issue_priorities: 項目優先權
 enumeration_doc_categories: 文件分類
 enumeration_activities: 活動 (time tracking)
+label_overall_activity: Overall activity
index a917f0da7a5f84ce54e18d690acbe899a4535724..39ed5fc44fdc98a765fd404d8e18a16de2910013 100644 (file)
@@ -614,3 +614,4 @@ default_activity_development: 开发
 enumeration_issue_priorities: 问题优先级
 enumeration_doc_categories: 文档类别
 enumeration_activities: 活动(时间跟踪)
+label_overall_activity: Overall activity
index b08e9076198a19db785eac22990a0ff33486383d..30c5572eda20daac52eba4051ec7905e75426737 100644 (file)
@@ -20,7 +20,7 @@ class ARCondition
   
   def initialize(condition=nil)
     @conditions = ['1=1']
-    @conditions.add(condition) if condition
+    add(condition) if condition
   end
   
   def add(condition)
index b3a472720e6d2d2938f1bd42e036f6bfcfbd1d27..26f8408d69c39c5f715f3e9c5f474145426a611f 100644 (file)
@@ -169,6 +169,7 @@ div#activity dd { margin-bottom: 1em; }
 div#activity dt { margin-bottom: 1px; }
 div#activity dt .time { color: #777; font-size: 80%; }
 div#activity dd .description { font-style: italic; }
+div#activity span.project:after { content: " -"; }
 
 div#roadmap fieldset.related-issues { margin-bottom: 1em; }
 div#roadmap fieldset.related-issues ul { margin-top: 0.3em; margin-bottom: 0.3em; }
index b3c662039371424939d88e1866780ba5eb346bf6..4f42d93c401cf8fba697903e390169f95a609315 100644 (file)
@@ -44,9 +44,9 @@ issues_003:
   start_date: <%= 1.day.from_now.to_date.to_s(:db) %>\r
   due_date: <%= 40.day.ago.to_date.to_s(:db) %>\r
 issues_004: \r
-  created_on: 2006-07-19 21:07:27 +02:00\r
+  created_on: <%= 5.days.ago.to_date.to_s(:db) %>\r
   project_id: 2\r
-  updated_on: 2006-07-19 21:07:27 +02:00\r
+  updated_on: <%= 2.days.ago.to_date.to_s(:db) %>\r
   priority_id: 4\r
   subject: Issue on project 2\r
   id: 4\r
@@ -57,4 +57,18 @@ issues_004:
   assigned_to_id: \r
   author_id: 2\r
   status_id: 1\r
+issues_005: \r
+  created_on: <%= 5.days.ago.to_date.to_s(:db) %>\r
+  project_id: 3\r
+  updated_on: <%= 2.days.ago.to_date.to_s(:db) %>\r
+  priority_id: 4\r
+  subject: Subproject issue\r
+  id: 5\r
+  fixed_version_id: \r
+  category_id: \r
+  description: This is an issue on a cookbook subproject\r
+  tracker_id: 1\r
+  assigned_to_id: \r
+  author_id: 2\r
+  status_id: 1\r
   \r
index 5bb2438ddd29e6aebf85ae4a87c4b302dec558f7..f82f376c14323de3d9e3077e78f64f6fcc56549c 100644 (file)
@@ -45,8 +45,8 @@ messages_004:
   parent_id: \r
   board_id: 1\r
 messages_005: \r
-  created_on: 2007-09-12 17:18:00 +02:00\r
-  updated_on: 2007-09-12 17:18:00 +02:00\r
+  created_on: <%= 3.days.ago.to_date.to_s(:db) %>\r
+  updated_on: <%= 3.days.ago.to_date.to_s(:db) %>\r
   subject: 'RE: post 2'\r
   id: 5\r
   replies_count: 0\r
index f610469df084e0f8ed5c7b7857803e59fd464402..75b4673a1fa3e5c6de0e001265b6aa31f60b04e2 100644 (file)
@@ -22,7 +22,8 @@ require 'projects_controller'
 class ProjectsController; def rescue_action(e) raise e end; end
 
 class ProjectsControllerTest < Test::Unit::TestCase
-  fixtures :projects, :versions, :users, :roles, :members, :issues, :journals, :journal_details, :trackers, :projects_trackers, :issue_statuses, :enabled_modules, :enumerations
+  fixtures :projects, :versions, :users, :roles, :members, :issues, :journals, :journal_details,
+           :trackers, :projects_trackers, :issue_statuses, :enabled_modules, :enumerations, :boards, :messages
 
   def setup
     @controller = ProjectsController.new
@@ -129,11 +130,15 @@ class ProjectsControllerTest < Test::Unit::TestCase
     assert assigns(:versions).include?(Version.find(1))
   end
 
-  def test_activity
-    get :activity, :id => 1
+  def test_project_activity
+    get :activity, :id => 1, :with_subprojects => 0
     assert_response :success
     assert_template 'activity'
     assert_not_nil assigns(:events_by_day)
+    assert_not_nil assigns(:events)
+
+    # subproject issue not included by default
+    assert !assigns(:events).include?(Issue.find(5))
     
     assert_tag :tag => "h3", 
                :content => /#{2.days.ago.to_date.day}/,
@@ -163,6 +168,53 @@ class ProjectsControllerTest < Test::Unit::TestCase
                }
   end
   
+  def test_activity_with_subprojects
+    get :activity, :id => 1, :with_subprojects => 1
+    assert_response :success
+    assert_template 'activity'
+    assert_not_nil assigns(:events)
+    
+    assert assigns(:events).include?(Issue.find(1))
+    assert !assigns(:events).include?(Issue.find(4))
+    # subproject issue
+    assert assigns(:events).include?(Issue.find(5))
+  end
+  
+  def test_global_activity_anonymous
+    get :activity
+    assert_response :success
+    assert_template 'activity'
+    assert_not_nil assigns(:events)
+    
+    assert assigns(:events).include?(Issue.find(1))
+    # Issue of a private project
+    assert !assigns(:events).include?(Issue.find(4))
+  end
+  
+  def test_global_activity_logged_user
+    @request.session[:user_id] = 2 # manager
+    get :activity
+    assert_response :success
+    assert_template 'activity'
+    assert_not_nil assigns(:events)
+    
+    assert assigns(:events).include?(Issue.find(1))
+    # Issue of a private project the user belongs to
+    assert assigns(:events).include?(Issue.find(4))
+  end
+
+  
+  def test_global_activity_with_all_types
+    get :activity, :show_issues => 1, :show_news => 1, :show_files => 1, :show_documents => 1, :show_changesets => 1, :show_wiki_pages => 1, :show_messages => 1
+    assert_response :success
+    assert_template 'activity'
+    assert_not_nil assigns(:events)
+    
+    assert assigns(:events).include?(Issue.find(1))
+    assert !assigns(:events).include?(Issue.find(4))
+    assert assigns(:events).include?(Message.find(5))
+  end
+
   def test_calendar
     get :calendar, :id => 1
     assert_response :success
index b05b7b09f05c455b5a9153641c5cfc32c921e08b..f7da6ecb587c357dea6f17a562a452c3b7e777b2 100644 (file)
@@ -126,13 +126,4 @@ class ProjectTest < Test::Unit::TestCase
     assert_equal [1, 2, 3], parent.rolled_up_trackers.collect(&:id)\r
     assert_equal [2, 3], child.rolled_up_trackers.collect(&:id)\r
   end\r
-\r
-  def test_issues_status_changes\r
-    journals = @ecookbook.issues_status_changes 3.days.ago.to_date, Date.today\r
-    assert_equal 1, journals.size\r
-    assert_kind_of Journal, journals.first\r
-    \r
-    journals = @ecookbook.issues_status_changes 30.days.ago.to_date, 10.days.ago.to_date\r
-    assert_equal 0, journals.size\r
-  end\r
 end\r