@@ -1,97 +1,97 | |||
|
1 | 1 | # Redmine - project management software |
|
2 | 2 | # Copyright (C) 2006-2011 Jean-Philippe Lang |
|
3 | 3 | # |
|
4 | 4 | # This program is free software; you can redistribute it and/or |
|
5 | 5 | # modify it under the terms of the GNU General Public License |
|
6 | 6 | # as published by the Free Software Foundation; either version 2 |
|
7 | 7 | # of the License, or (at your option) any later version. |
|
8 | 8 | # |
|
9 | 9 | # This program is distributed in the hope that it will be useful, |
|
10 | 10 | # but WITHOUT ANY WARRANTY; without even the implied warranty of |
|
11 | 11 | # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
|
12 | 12 | # GNU General Public License for more details. |
|
13 | 13 | # |
|
14 | 14 | # You should have received a copy of the GNU General Public License |
|
15 | 15 | # along with this program; if not, write to the Free Software |
|
16 | 16 | # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. |
|
17 | 17 | |
|
18 | 18 | class IssueRelationsController < ApplicationController |
|
19 | 19 | before_filter :find_issue, :find_project_from_association, :authorize, :only => [:index, :create] |
|
20 | 20 | before_filter :find_relation, :except => [:index, :create] |
|
21 | 21 | |
|
22 | 22 | accept_api_auth :index, :show, :create, :destroy |
|
23 | 23 | |
|
24 | 24 | def index |
|
25 | 25 | @relations = @issue.relations |
|
26 | 26 | |
|
27 | 27 | respond_to do |format| |
|
28 | 28 | format.html { render :nothing => true } |
|
29 | 29 | format.api |
|
30 | 30 | end |
|
31 | 31 | end |
|
32 | 32 | |
|
33 | 33 | def show |
|
34 | 34 | raise Unauthorized unless @relation.visible? |
|
35 | 35 | |
|
36 | 36 | respond_to do |format| |
|
37 | 37 | format.html { render :nothing => true } |
|
38 | 38 | format.api |
|
39 | 39 | end |
|
40 | 40 | end |
|
41 | 41 | |
|
42 | 42 | verify :method => :post, :only => :create, :render => {:nothing => true, :status => :method_not_allowed } |
|
43 | 43 | def create |
|
44 | 44 | @relation = IssueRelation.new(params[:relation]) |
|
45 | 45 | @relation.issue_from = @issue |
|
46 | 46 | if params[:relation] && m = params[:relation][:issue_to_id].to_s.match(/^#?(\d+)$/) |
|
47 | 47 | @relation.issue_to = Issue.visible.find_by_id(m[1].to_i) |
|
48 | 48 | end |
|
49 | 49 | saved = @relation.save |
|
50 | 50 | |
|
51 | 51 | respond_to do |format| |
|
52 | 52 | format.html { redirect_to :controller => 'issues', :action => 'show', :id => @issue } |
|
53 | 53 | format.js do |
|
54 | 54 | @relations = @issue.relations.select {|r| r.other_issue(@issue) && r.other_issue(@issue).visible? } |
|
55 | 55 | render :update do |page| |
|
56 | 56 | page.replace_html "relations", :partial => 'issues/relations' |
|
57 | 57 | if @relation.errors.empty? |
|
58 | 58 | page << "$('relation_delay').value = ''" |
|
59 | 59 | page << "$('relation_issue_to_id').value = ''" |
|
60 | 60 | end |
|
61 | 61 | end |
|
62 | 62 | end |
|
63 | 63 | format.api { |
|
64 | 64 | if saved |
|
65 | 65 | render :action => 'show', :status => :created, :location => relation_url(@relation) |
|
66 | 66 | else |
|
67 | 67 | render_validation_errors(@relation) |
|
68 | 68 | end |
|
69 | 69 | } |
|
70 | 70 | end |
|
71 | 71 | end |
|
72 | 72 | |
|
73 | 73 | verify :method => :delete, :only => :destroy, :render => {:nothing => true, :status => :method_not_allowed } |
|
74 | 74 | def destroy |
|
75 | 75 | raise Unauthorized unless @relation.deletable? |
|
76 | 76 | @relation.destroy |
|
77 | 77 | |
|
78 | 78 | respond_to do |format| |
|
79 | format.html { redirect_to :controller => 'issues', :action => 'show', :id => @issue } | |
|
79 | format.html { redirect_to issue_path } # TODO : does this really work since @issue is always nil? What is it useful to? | |
|
80 | 80 | format.js { render(:update) {|page| page.remove "relation-#{@relation.id}"} } |
|
81 | 81 | format.api { head :ok } |
|
82 | 82 | end |
|
83 | 83 | end |
|
84 | 84 | |
|
85 | 85 | private |
|
86 | 86 | def find_issue |
|
87 | 87 | @issue = @object = Issue.find(params[:issue_id]) |
|
88 | 88 | rescue ActiveRecord::RecordNotFound |
|
89 | 89 | render_404 |
|
90 | 90 | end |
|
91 | 91 | |
|
92 | 92 | def find_relation |
|
93 | 93 | @relation = IssueRelation.find(params[:id]) |
|
94 | 94 | rescue ActiveRecord::RecordNotFound |
|
95 | 95 | render_404 |
|
96 | 96 | end |
|
97 | 97 | end |
@@ -1,36 +1,36 | |||
|
1 | 1 | <div class="contextual"> |
|
2 | 2 | <%= link_to l(:label_personalize_page), :action => 'page_layout' %> |
|
3 | 3 | </div> |
|
4 | 4 | |
|
5 | 5 | <h2><%=l(:label_my_page)%></h2> |
|
6 | 6 | |
|
7 | 7 | <div id="list-top"> |
|
8 | 8 | <% @blocks['top'].each do |b| |
|
9 | 9 | next unless MyController::BLOCKS.keys.include? b %> |
|
10 | 10 | <div class="mypage-box"> |
|
11 | 11 | <%= render :partial => "my/blocks/#{b}", :locals => { :user => @user } %> |
|
12 | 12 | </div> |
|
13 | 13 | <% end if @blocks['top'] %> |
|
14 | 14 | </div> |
|
15 | 15 | |
|
16 | 16 | <div id="list-left" class="splitcontentleft"> |
|
17 | 17 | <% @blocks['left'].each do |b| |
|
18 | 18 | next unless MyController::BLOCKS.keys.include? b %> |
|
19 | 19 | <div class="mypage-box"> |
|
20 | 20 | <%= render :partial => "my/blocks/#{b}", :locals => { :user => @user } %> |
|
21 | 21 | </div> |
|
22 | 22 | <% end if @blocks['left'] %> |
|
23 | 23 | </div> |
|
24 | 24 | |
|
25 | 25 | <div id="list-right" class="splitcontentright"> |
|
26 | 26 | <% @blocks['right'].each do |b| |
|
27 | 27 | next unless MyController::BLOCKS.keys.include? b %> |
|
28 | 28 | <div class="mypage-box"> |
|
29 | 29 | <%= render :partial => "my/blocks/#{b}", :locals => { :user => @user } %> |
|
30 | 30 | </div> |
|
31 | 31 | <% end if @blocks['right'] %> |
|
32 | 32 | </div> |
|
33 | 33 | |
|
34 |
<%= context_menu |
|
|
34 | <%= context_menu issues_context_menu_path %> | |
|
35 | 35 | |
|
36 | 36 | <% html_title(l(:label_my_page)) -%> |
@@ -1,26 +1,26 | |||
|
1 | 1 | <% content_for :header_tags do %> |
|
2 | 2 | <%= auto_discovery_link_tag(:atom, {:action => 'index', :format => 'atom', :key => User.current.rss_key}) %> |
|
3 | 3 | <% end %> |
|
4 | 4 | |
|
5 | 5 | <div class="contextual"> |
|
6 | 6 | <%= link_to(l(:label_project_new), {:controller => 'projects', :action => 'new'}, :class => 'icon icon-add') + ' |' if User.current.allowed_to?(:add_project, nil, :global => true) %> |
|
7 | 7 | <%= link_to(l(:label_issue_view_all), { :controller => 'issues' }) + ' |' if User.current.allowed_to?(:view_issues, nil, :global => true) %> |
|
8 |
<%= link_to(l(:label_overall_spent_time), |
|
|
8 | <%= link_to(l(:label_overall_spent_time), time_entries_path) + ' |' if User.current.allowed_to?(:view_time_entries, nil, :global => true) %> | |
|
9 | 9 | <%= link_to l(:label_overall_activity), { :controller => 'activities', :action => 'index' }%> |
|
10 | 10 | </div> |
|
11 | 11 | |
|
12 | 12 | <h2><%=l(:label_project_plural)%></h2> |
|
13 | 13 | |
|
14 | 14 | <%= render_project_hierarchy(@projects)%> |
|
15 | 15 | |
|
16 | 16 | <% if User.current.logged? %> |
|
17 | 17 | <p style="text-align:right;"> |
|
18 | 18 | <span class="my-project"><%= l(:label_my_projects) %></span> |
|
19 | 19 | </p> |
|
20 | 20 | <% end %> |
|
21 | 21 | |
|
22 | 22 | <% other_formats_links do |f| %> |
|
23 | 23 | <%= f.link_to 'Atom', :url => {:key => User.current.rss_key} %> |
|
24 | 24 | <% end %> |
|
25 | 25 | |
|
26 | 26 | <% html_title(l(:label_project_plural)) -%> |
@@ -1,191 +1,282 | |||
|
1 | 1 | ActionController::Routing::Routes.draw do |map| |
|
2 | 2 | # Add your own custom routes here. |
|
3 | 3 | # The priority is based upon order of creation: first created -> highest priority. |
|
4 | 4 | |
|
5 | 5 | # Here's a sample route: |
|
6 | 6 | # map.connect 'products/:id', :controller => 'catalog', :action => 'view' |
|
7 | 7 | # Keep in mind you can assign values other than :controller and :action |
|
8 | 8 | |
|
9 | map.home '', :controller => 'welcome' | |
|
9 | map.home '', :controller => 'welcome', :conditions => {:method => :get} | |
|
10 | 10 | |
|
11 | map.signin 'login', :controller => 'account', :action => 'login' | |
|
12 | map.signout 'logout', :controller => 'account', :action => 'logout' | |
|
11 | map.signin 'login', :controller => 'account', :action => 'login', :conditions => {:method => [:get, :post]} | |
|
12 | map.signout 'logout', :controller => 'account', :action => 'logout', :conditions => {:method => :get} | |
|
13 | map.connect 'account/register', :controller => 'account', :action => 'register', :conditions => {:method => [:get, :post]} | |
|
14 | map.connect 'account/lost_password', :controller => 'account', :action => 'lost_password', :conditions => {:method => [:get, :post]} | |
|
15 | map.connect 'account/login', :controller => 'account', :action => 'login', :conditions => {:method => [:get, :post]} | |
|
16 | map.connect 'account/logout', :controller => 'account', :action => 'logout', :conditions => {:method => :get} | |
|
17 | map.connect 'account/activate', :controller => 'account', :action => 'activate', :conditions => {:method => :get} | |
|
13 | 18 | |
|
14 | 19 | map.connect 'roles/workflow/:id/:role_id/:tracker_id', :controller => 'roles', :action => 'workflow' |
|
15 | map.connect 'help/:ctrl/:page', :controller => 'help' | |
|
16 | 20 | |
|
21 | map.connect 'help/:ctrl/:page', :controller => 'help', :conditions => {:method => :get} | |
|
22 | ||
|
23 | map.connect '/time_entries/destroy', | |
|
24 | :controller => 'timelog', :action => 'destroy', :conditions => { :method => :delete } | |
|
17 | 25 | map.time_entries_context_menu '/time_entries/context_menu', |
|
18 | 26 | :controller => 'context_menus', :action => 'time_entries' |
|
19 | 27 | |
|
20 | 28 | map.resources :time_entries, :controller => 'timelog', :collection => {:report => :get, :bulk_edit => :get, :bulk_update => :post} |
|
21 | 29 | |
|
22 | 30 | map.connect 'projects/:id/wiki', :controller => 'wikis', :action => 'edit', :conditions => {:method => :post} |
|
23 | map.connect 'projects/:id/wiki/destroy', :controller => 'wikis', :action => 'destroy', :conditions => {:method => :get} | |
|
24 | map.connect 'projects/:id/wiki/destroy', :controller => 'wikis', :action => 'destroy', :conditions => {:method => :post} | |
|
31 | map.connect 'projects/:id/wiki/destroy', :controller => 'wikis', :action => 'destroy', :conditions => {:method => [:get, :post]} | |
|
25 | 32 | |
|
26 | 33 | map.with_options :controller => 'messages' do |messages_routes| |
|
27 | 34 | messages_routes.with_options :conditions => {:method => :get} do |messages_views| |
|
28 | 35 | messages_views.connect 'boards/:board_id/topics/new', :action => 'new' |
|
29 | 36 | messages_views.connect 'boards/:board_id/topics/:id', :action => 'show' |
|
37 | messages_views.connect 'boards/:board_id/topics/quote/:id', :action => 'quote' | |
|
30 | 38 | messages_views.connect 'boards/:board_id/topics/:id/edit', :action => 'edit' |
|
31 | 39 | end |
|
32 | 40 | messages_routes.with_options :conditions => {:method => :post} do |messages_actions| |
|
33 | 41 | messages_actions.connect 'boards/:board_id/topics/new', :action => 'new' |
|
42 | messages_actions.connect 'boards/:board_id/topics/preview', :action => 'preview' | |
|
34 | 43 | messages_actions.connect 'boards/:board_id/topics/:id/replies', :action => 'reply' |
|
35 | 44 | messages_actions.connect 'boards/:board_id/topics/:id/:action', :action => /edit|destroy/ |
|
36 | 45 | end |
|
37 | 46 | end |
|
38 | 47 | |
|
39 | 48 | map.resources :issue_moves, :only => [:new, :create], :path_prefix => '/issues', :as => 'move' |
|
40 | 49 | map.resources :queries, :except => [:show] |
|
41 | 50 | |
|
42 | 51 | # Misc issue routes. TODO: move into resources |
|
43 | 52 | map.auto_complete_issues '/issues/auto_complete', :controller => 'auto_completes', :action => 'issues', :conditions => { :method => :get } |
|
44 | 53 | map.preview_issue '/issues/preview/:id', :controller => 'previews', :action => 'issue' # TODO: would look nicer as /issues/:id/preview |
|
45 | 54 | map.issues_context_menu '/issues/context_menu', :controller => 'context_menus', :action => 'issues' |
|
46 | 55 | map.issue_changes '/issues/changes', :controller => 'journals', :action => 'index' |
|
47 | 56 | map.quoted_issue '/issues/:id/quoted', :controller => 'journals', :action => 'new', :id => /\d+/, :conditions => { :method => :post } |
|
57 | map.connect '/journals/diff', :controller => 'journals', :action => 'diff' | |
|
58 | map.connect '/journals/edit/:id', :controller => 'journals', :action => 'edit', :id => /\d+/, :conditions => { :method => [:get, :post] } | |
|
48 | 59 | |
|
49 | 60 | map.with_options :controller => 'gantts', :action => 'show' do |gantts_routes| |
|
50 | 61 | gantts_routes.connect '/projects/:project_id/issues/gantt' |
|
51 | 62 | gantts_routes.connect '/projects/:project_id/issues/gantt.:format' |
|
52 | 63 | gantts_routes.connect '/issues/gantt.:format' |
|
53 | 64 | end |
|
54 | 65 | |
|
55 | 66 | map.with_options :controller => 'calendars', :action => 'show' do |calendars_routes| |
|
56 | 67 | calendars_routes.connect '/projects/:project_id/issues/calendar' |
|
57 | 68 | calendars_routes.connect '/issues/calendar' |
|
58 | 69 | end |
|
59 | 70 | |
|
60 | 71 | map.with_options :controller => 'reports', :conditions => {:method => :get} do |reports| |
|
61 | 72 | reports.connect 'projects/:id/issues/report', :action => 'issue_report' |
|
62 | 73 | reports.connect 'projects/:id/issues/report/:detail', :action => 'issue_report_details' |
|
63 | 74 | end |
|
64 | 75 | |
|
76 | map.connect 'my/account', :controller => 'my', :action => 'account', :conditions => {:method => [:get, :post]} | |
|
77 | map.connect 'my/page', :controller => 'my', :action => 'page', :conditions => {:method => :get} | |
|
78 | map.connect 'my', :controller => 'my', :action => 'index', :conditions => {:method => :get} # Redirects to my/page | |
|
79 | map.connect 'my/reset_rss_key', :controller => 'my', :action => 'reset_rss_key', :conditions => {:method => :post} | |
|
80 | map.connect 'my/reset_api_key', :controller => 'my', :action => 'reset_api_key', :conditions => {:method => :post} | |
|
81 | map.connect 'my/password', :controller => 'my', :action => 'password', :conditions => {:method => [:get, :post]} | |
|
82 | map.connect 'my/page_layout', :controller => 'my', :action => 'page_layout', :conditions => {:method => :get} | |
|
83 | map.connect 'my/add_block', :controller => 'my', :action => 'add_block', :conditions => {:method => :post} | |
|
84 | map.connect 'my/remove_block', :controller => 'my', :action => 'remove_block', :conditions => {:method => :post} | |
|
85 | map.connect 'my/order_blocks', :controller => 'my', :action => 'order_blocks', :conditions => {:method => :post} | |
|
86 | ||
|
65 | 87 | map.resources :issues, :collection => {:bulk_edit => :get, :bulk_update => :post} do |issues| |
|
66 | 88 | issues.resources :time_entries, :controller => 'timelog', :collection => {:report => :get} |
|
67 | 89 | issues.resources :relations, :shallow => true, :controller => 'issue_relations', :only => [:index, :show, :create, :destroy] |
|
68 | 90 | end |
|
69 | 91 | # Bulk deletion |
|
70 | 92 | map.connect '/issues', :controller => 'issues', :action => 'destroy', :conditions => {:method => :delete} |
|
71 | 93 | |
|
72 | map.connect 'projects/:id/members/new', :controller => 'members', :action => 'new' | |
|
94 | map.connect 'projects/:id/members/new', :controller => 'members', :action => 'new', :conditions => { :method => :post } | |
|
95 | map.connect 'members/edit/:id', :controller => 'members', :action => 'edit', :id => /\d+/, :conditions => { :method => :post } | |
|
96 | map.connect 'members/destroy/:id', :controller => 'members', :action => 'destroy', :id => /\d+/, :conditions => { :method => :post } | |
|
97 | map.connect 'members/autocomplete_for_member/:id', :controller => 'members', :action => 'autocomplete_for_member', :conditions => { :method => :post } | |
|
73 | 98 | |
|
74 | 99 | map.resources :users |
|
75 | 100 | map.with_options :controller => 'users' do |users| |
|
76 | 101 | users.user_memberships 'users/:id/memberships', :action => 'edit_membership', :conditions => {:method => :post} |
|
77 | 102 | users.user_membership 'users/:id/memberships/:membership_id', :action => 'edit_membership', :conditions => {:method => :put} |
|
78 | 103 | users.connect 'users/:id/memberships/:membership_id', :action => 'destroy_membership', :conditions => {:method => :delete} |
|
79 | 104 | end |
|
80 | 105 | |
|
81 | 106 | # For nice "roadmap" in the url for the index action |
|
82 | 107 | map.connect 'projects/:project_id/roadmap', :controller => 'versions', :action => 'index' |
|
83 | 108 | |
|
84 | 109 | map.all_news 'news', :controller => 'news', :action => 'index' |
|
85 | 110 | map.formatted_all_news 'news.:format', :controller => 'news', :action => 'index' |
|
86 | 111 | map.preview_news '/news/preview', :controller => 'previews', :action => 'news' |
|
87 | 112 | map.connect 'news/:id/comments', :controller => 'comments', :action => 'create', :conditions => {:method => :post} |
|
88 | 113 | map.connect 'news/:id/comments/:comment_id', :controller => 'comments', :action => 'destroy', :conditions => {:method => :delete} |
|
89 | 114 | |
|
115 | map.connect 'watchers/new', :controller=> 'watchers', :action => 'new', :conditions => {:method => [:get, :post]} | |
|
116 | map.connect 'watchers/destroy', :controller=> 'watchers', :action => 'destroy', :conditions => {:method => :post} | |
|
117 | map.connect 'watchers/watch', :controller=> 'watchers', :action => 'watch', :conditions => {:method => :post} | |
|
118 | map.connect 'watchers/unwatch', :controller=> 'watchers', :action => 'unwatch', :conditions => {:method => :post} | |
|
119 | ||
|
90 | 120 | map.resources :projects, :member => { |
|
91 | 121 | :copy => [:get, :post], |
|
92 | 122 | :settings => :get, |
|
93 | 123 | :modules => :post, |
|
94 | 124 | :archive => :post, |
|
95 | 125 | :unarchive => :post |
|
96 | 126 | } do |project| |
|
97 | 127 | project.resource :project_enumerations, :as => 'enumerations', :only => [:update, :destroy] |
|
98 | 128 | project.resources :issues, :only => [:index, :new, :create] do |issues| |
|
99 | 129 | issues.resources :time_entries, :controller => 'timelog', :collection => {:report => :get} |
|
100 | 130 | end |
|
101 | 131 | project.resources :files, :only => [:index, :new, :create] |
|
102 | 132 | project.resources :versions, :shallow => true, :collection => {:close_completed => :put}, :member => {:status_by => :post} |
|
103 | 133 | project.resources :news, :shallow => true |
|
104 | 134 | project.resources :time_entries, :controller => 'timelog', :path_prefix => 'projects/:project_id', :collection => {:report => :get} |
|
105 | 135 | project.resources :queries, :only => [:new, :create] |
|
106 | 136 | project.resources :issue_categories, :shallow => true |
|
107 | 137 | project.resources :documents, :shallow => true, :member => {:add_attachment => :post} |
|
108 | 138 | project.resources :boards |
|
109 | 139 | |
|
110 | 140 | project.wiki_start_page 'wiki', :controller => 'wiki', :action => 'show', :conditions => {:method => :get} |
|
111 | 141 | project.wiki_index 'wiki/index', :controller => 'wiki', :action => 'index', :conditions => {:method => :get} |
|
112 | 142 | project.wiki_diff 'wiki/:id/diff/:version', :controller => 'wiki', :action => 'diff', :version => nil |
|
113 | 143 | project.wiki_diff 'wiki/:id/diff/:version/vs/:version_from', :controller => 'wiki', :action => 'diff' |
|
114 | 144 | project.wiki_annotate 'wiki/:id/annotate/:version', :controller => 'wiki', :action => 'annotate' |
|
115 | 145 | project.resources :wiki, :except => [:new, :create], :member => { |
|
116 | 146 | :rename => [:get, :post], |
|
117 | 147 | :history => :get, |
|
118 | 148 | :preview => :any, |
|
119 | 149 | :protect => :post, |
|
120 | 150 | :add_attachment => :post |
|
121 | 151 | }, :collection => { |
|
122 | 152 | :export => :get, |
|
123 | 153 | :date_index => :get |
|
124 | 154 | } |
|
125 | 155 | |
|
126 | 156 | end |
|
127 | 157 | |
|
128 | 158 | # TODO: port to be part of the resources route(s) |
|
129 | 159 | map.with_options :controller => 'projects' do |project_mapper| |
|
130 | 160 | project_mapper.with_options :conditions => {:method => :get} do |project_views| |
|
131 | 161 | project_views.connect 'projects/:id/settings/:tab', :controller => 'projects', :action => 'settings' |
|
132 | 162 | project_views.connect 'projects/:project_id/issues/:copy_from/copy', :controller => 'issues', :action => 'new' |
|
133 | 163 | end |
|
134 | 164 | end |
|
135 | 165 | |
|
136 | 166 | map.with_options :controller => 'activities', :action => 'index', :conditions => {:method => :get} do |activity| |
|
137 | 167 | activity.connect 'projects/:id/activity' |
|
138 | 168 | activity.connect 'projects/:id/activity.:format' |
|
139 | 169 | activity.connect 'activity', :id => nil |
|
140 | 170 | activity.connect 'activity.:format', :id => nil |
|
141 | 171 | end |
|
142 | 172 | |
|
143 | 173 | map.with_options :controller => 'repositories' do |repositories| |
|
144 | 174 | repositories.with_options :conditions => {:method => :get} do |repository_views| |
|
145 | 175 | repository_views.connect 'projects/:id/repository', :action => 'show' |
|
146 | 176 | repository_views.connect 'projects/:id/repository/edit', :action => 'edit' |
|
147 | 177 | repository_views.connect 'projects/:id/repository/statistics', :action => 'stats' |
|
148 | 178 | repository_views.connect 'projects/:id/repository/revisions', :action => 'revisions' |
|
149 | 179 | repository_views.connect 'projects/:id/repository/revisions.:format', :action => 'revisions' |
|
150 | 180 | repository_views.connect 'projects/:id/repository/revisions/:rev', :action => 'revision' |
|
151 | 181 | repository_views.connect 'projects/:id/repository/revisions/:rev/diff', :action => 'diff' |
|
152 | 182 | repository_views.connect 'projects/:id/repository/revisions/:rev/diff.:format', :action => 'diff' |
|
153 | 183 | repository_views.connect 'projects/:id/repository/revisions/:rev/raw/*path', :action => 'entry', :format => 'raw', :requirements => { :rev => /[a-z0-9\.\-_]+/ } |
|
154 | 184 | repository_views.connect 'projects/:id/repository/revisions/:rev/:action/*path', :requirements => { :rev => /[a-z0-9\.\-_]+/ } |
|
155 | 185 | repository_views.connect 'projects/:id/repository/raw/*path', :action => 'entry', :format => 'raw' |
|
156 | # TODO: why the following route is required? | |
|
186 | repository_views.connect 'projects/:id/repository/browse/*path', :action => 'browse' | |
|
157 | 187 | repository_views.connect 'projects/:id/repository/entry/*path', :action => 'entry' |
|
158 |
repository_views.connect 'projects/:id/repository/ |
|
|
188 | repository_views.connect 'projects/:id/repository/changes/*path', :action => 'changes' | |
|
189 | repository_views.connect 'projects/:id/repository/annotate/*path', :action => 'annotate' | |
|
190 | repository_views.connect 'projects/:id/repository/diff/*path', :action => 'diff' | |
|
191 | repository_views.connect 'projects/:id/repository/graph', :action => 'graph' | |
|
192 | # repository_views.connect 'projects/:id/repository/:action/*path' | |
|
159 | 193 | end |
|
160 | 194 | |
|
161 | repositories.connect 'projects/:id/repository/:action', :conditions => {:method => :post} | |
|
195 | repositories.connect 'projects/:id/repository/revision', :action => 'revision', :conditions => {:method => [:get, :post]} | |
|
196 | repositories.connect 'projects/:id/repository/committers', :action => 'committers', :conditions => {:method => [:get, :post]} | |
|
197 | repositories.connect 'projects/:id/repository/edit', :action => 'edit', :conditions => {:method => :post} | |
|
198 | repositories.connect 'projects/:id/repository/destroy', :action => 'destroy', :conditions => {:method => :post} | |
|
199 | # repositories.connect 'projects/:id/repository/:action', :conditions => {:method => :post} | |
|
162 | 200 | end |
|
163 | ||
|
201 | ||
|
164 | 202 | map.resources :attachments, :only => [:show, :destroy] |
|
165 | 203 | # additional routes for having the file name at the end of url |
|
166 | 204 | map.connect 'attachments/:id/:filename', :controller => 'attachments', :action => 'show', :id => /\d+/, :filename => /.*/ |
|
167 | 205 | map.connect 'attachments/download/:id/:filename', :controller => 'attachments', :action => 'download', :id => /\d+/, :filename => /.*/ |
|
206 | map.connect 'attachments/download/:id', :controller => 'attachments', :action => 'download', :id => /\d+/ | |
|
168 | 207 | |
|
169 | 208 | map.resources :groups, :member => {:autocomplete_for_user => :get} |
|
170 | 209 | map.group_users 'groups/:id/users', :controller => 'groups', :action => 'add_users', :id => /\d+/, :conditions => {:method => :post} |
|
171 | 210 | map.group_user 'groups/:id/users/:user_id', :controller => 'groups', :action => 'remove_user', :id => /\d+/, :conditions => {:method => :delete} |
|
211 | map.connect 'groups/add_users/:id', :controller => 'groups', :action => 'add_users', :id => /\d+/, :conditions => {:method => :post} | |
|
212 | map.connect 'groups/remove_user/:id', :controller => 'groups', :action => 'remove_user', :id => /\d+/, :conditions => {:method => :post} | |
|
213 | map.connect 'groups/destroy_membership/:id', :controller => 'groups', :action => 'destroy_membership', :id => /\d+/, :conditions => {:method => :post} | |
|
214 | map.connect 'groups/edit_membership/:id', :controller => 'groups', :action => 'edit_membership', :id => /\d+/, :conditions => {:method => :post} | |
|
172 | 215 | |
|
173 | 216 | map.resources :trackers, :except => :show |
|
174 | 217 | map.resources :issue_statuses, :except => :show, :collection => {:update_issue_done_ratio => :post} |
|
175 | 218 | map.resources :custom_fields, :except => :show |
|
176 | 219 | map.resources :roles, :except => :show, :collection => {:permissions => [:get, :post]} |
|
177 | 220 | |
|
178 | #left old routes at the bottom for backwards compat | |
|
179 | map.connect 'boards/:board_id/topics/:action/:id', :controller => 'messages' | |
|
221 | map.connect 'custom_fields', :controller => 'custom_fields', :action => 'index', :conditions => {:method => :get} | |
|
222 | map.connect 'custom_fields/new', :controller => 'custom_fields', :action => 'new', :conditions => {:method => [:get, :post]} | |
|
223 | map.connect 'custom_fields/edit/:id', :controller => 'custom_fields', :action => 'edit', :id => /\d+/, :conditions => {:method => [:get, :post]} | |
|
224 | map.connect 'custom_fields/destroy/:id', :controller => 'custom_fields', :action => 'destroy', :id => /\d+/, :conditions => {:method => :post} | |
|
225 | ||
|
226 | map.connect 'search', :controller => 'search', :action => 'index', :conditions => {:method => :get} | |
|
227 | ||
|
228 | map.connect 'mail_handler', :controller => 'mail_handler', :action => 'index', :conditions => {:method => :post} | |
|
229 | ||
|
230 | map.connect 'admin', :controller => 'admin', :action => 'index', :conditions => {:method => :get} | |
|
231 | map.connect 'admin/projects', :controller => 'admin', :action => 'projects', :conditions => {:method => :get} | |
|
232 | map.connect 'admin/plugins', :controller => 'admin', :action => 'plugins', :conditions => {:method => :get} | |
|
233 | map.connect 'admin/info', :controller => 'admin', :action => 'info', :conditions => {:method => :get} | |
|
234 | map.connect 'admin/test_email', :controller => 'admin', :action => 'test_email', :conditions => {:method => :get} | |
|
235 | map.connect 'admin/default_configuration', :controller => 'admin', :action => 'default_configuration', :conditions => {:method => :post} | |
|
236 | ||
|
237 | # Used by AuthSourcesControllerTest | |
|
238 | # TODO : refactor *AuthSourcesController to remove these routes | |
|
239 | map.connect 'auth_sources', :controller => 'auth_sources', :action => 'index', :conditions => {:method => :get} | |
|
240 | map.connect 'auth_sources/new', :controller => 'auth_sources', :action => 'new', :conditions => {:method => :get} | |
|
241 | map.connect 'auth_sources/create', :controller => 'auth_sources', :action => 'create', :conditions => {:method => :post} | |
|
242 | map.connect 'auth_sources/destroy/:id', :controller => 'auth_sources', :action => 'destroy', :id => /\d+/, :conditions => {:method => :post} | |
|
243 | map.connect 'auth_sources/test_connection/:id', :controller => 'auth_sources', :action => 'test_connection', :conditions => {:method => :get} | |
|
244 | map.connect 'auth_sources/edit/:id', :controller => 'auth_sources', :action => 'edit', :id => /\d+/, :conditions => {:method => :get} | |
|
245 | map.connect 'auth_sources/update/:id', :controller => 'auth_sources', :action => 'update', :id => /\d+/, :conditions => {:method => :post} | |
|
246 | ||
|
247 | map.connect 'ldap_auth_sources', :controller => 'ldap_auth_sources', :action => 'index', :conditions => {:method => :get} | |
|
248 | map.connect 'ldap_auth_sources/new', :controller => 'ldap_auth_sources', :action => 'new', :conditions => {:method => :get} | |
|
249 | map.connect 'ldap_auth_sources/create', :controller => 'ldap_auth_sources', :action => 'create', :conditions => {:method => :post} | |
|
250 | map.connect 'ldap_auth_sources/destroy/:id', :controller => 'ldap_auth_sources', :action => 'destroy', :id => /\d+/, :conditions => {:method => :post} | |
|
251 | map.connect 'ldap_auth_sources/test_connection/:id', :controller => 'ldap_auth_sources', :action => 'test_connection', :conditions => {:method => :get} | |
|
252 | map.connect 'ldap_auth_sources/edit/:id', :controller => 'ldap_auth_sources', :action => 'edit', :id => /\d+/, :conditions => {:method => :get} | |
|
253 | map.connect 'ldap_auth_sources/update/:id', :controller => 'ldap_auth_sources', :action => 'update', :id => /\d+/, :conditions => {:method => :post} | |
|
254 | map.connect 'workflows', :controller => 'workflows', :action => 'index', :conditions => {:method => :get} | |
|
255 | map.connect 'workflows/edit', :controller => 'workflows', :action => 'edit', :conditions => {:method => [:get, :post]} | |
|
256 | map.connect 'workflows/copy', :controller => 'workflows', :action => 'copy', :conditions => {:method => [:get, :post]} | |
|
257 | map.connect 'enumerations', :controller => 'enumerations', :action => 'index', :conditions => {:method => :get} | |
|
258 | map.connect 'enumerations/new', :controller => 'enumerations', :action => 'new', :conditions => {:method => :get} | |
|
259 | map.connect 'enumerations/create', :controller => 'enumerations', :action => 'create', :conditions => {:method => :post} | |
|
260 | map.connect 'enumerations/edit/:id', :controller => 'enumerations', :action => 'edit', :id => /\d+/, :conditions => {:method => :get} | |
|
261 | map.connect 'enumerations/update/:id', :controller => 'enumerations', :action => 'update', :id => /\d+/, :conditions => {:method => :post} | |
|
262 | map.connect 'enumerations/destroy/:id', :controller => 'enumerations', :action => 'destroy', :id => /\d+/, :conditions => {:method => :post} | |
|
263 | map.connect 'settings', :controller => 'settings', :action => 'index', :conditions => {:method => :get} | |
|
264 | map.connect 'settings/edit', :controller => 'settings', :action => 'edit', :conditions => {:method => [:get, :post]} | |
|
265 | map.connect 'settings/plugin/:id', :controller => 'settings', :action => 'plugin', :conditions => {:method => [:get, :post]} | |
|
180 | 266 | |
|
181 | 267 | map.with_options :controller => 'sys' do |sys| |
|
182 | 268 | sys.connect 'sys/projects.:format', :action => 'projects', :conditions => {:method => :get} |
|
183 | 269 | sys.connect 'sys/projects/:id/repository.:format', :action => 'create_project_repository', :conditions => {:method => :post} |
|
270 | sys.connect 'sys/fetch_changesets', :action => 'fetch_changesets', :conditions => {:method => :get} | |
|
184 | 271 | end |
|
185 | 272 | |
|
273 | #left old routes at the bottom for backwards compat | |
|
274 | # map.connect 'boards/:board_id/topics/:action/:id', :controller => 'messages' | |
|
275 | ||
|
186 | 276 | # Install the default route as the lowest priority. |
|
187 | map.connect ':controller/:action/:id' | |
|
188 | map.connect 'robots.txt', :controller => 'welcome', :action => 'robots' | |
|
277 | # map.connect ':controller/:action/:id' | |
|
278 | map.connect 'robots.txt', :controller => 'welcome', :action => 'robots', :conditions => {:method => :get} | |
|
279 | ||
|
189 | 280 | # Used for OpenID |
|
190 | 281 | map.root :controller => 'account', :action => 'login' |
|
191 | 282 | end |
@@ -1,545 +1,538 | |||
|
1 | 1 | # Redmine - project management software |
|
2 | 2 | # Copyright (C) 2006-2011 Jean-Philippe Lang |
|
3 | 3 | # |
|
4 | 4 | # This program is free software; you can redistribute it and/or |
|
5 | 5 | # modify it under the terms of the GNU General Public License |
|
6 | 6 | # as published by the Free Software Foundation; either version 2 |
|
7 | 7 | # of the License, or (at your option) any later version. |
|
8 | 8 | # |
|
9 | 9 | # This program is distributed in the hope that it will be useful, |
|
10 | 10 | # but WITHOUT ANY WARRANTY; without even the implied warranty of |
|
11 | 11 | # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
|
12 | 12 | # GNU General Public License for more details. |
|
13 | 13 | # |
|
14 | 14 | # You should have received a copy of the GNU General Public License |
|
15 | 15 | # along with this program; if not, write to the Free Software |
|
16 | 16 | # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. |
|
17 | 17 | |
|
18 | 18 | require File.expand_path('../../test_helper', __FILE__) |
|
19 | 19 | require 'projects_controller' |
|
20 | 20 | |
|
21 | 21 | # Re-raise errors caught by the controller. |
|
22 | 22 | class ProjectsController; def rescue_action(e) raise e end; end |
|
23 | 23 | |
|
24 | 24 | class ProjectsControllerTest < ActionController::TestCase |
|
25 | 25 | fixtures :projects, :versions, :users, :roles, :members, :member_roles, :issues, :journals, :journal_details, |
|
26 | 26 | :trackers, :projects_trackers, :issue_statuses, :enabled_modules, :enumerations, :boards, :messages, |
|
27 | 27 | :attachments, :custom_fields, :custom_values, :time_entries |
|
28 | 28 | |
|
29 | 29 | def setup |
|
30 | 30 | @controller = ProjectsController.new |
|
31 | 31 | @request = ActionController::TestRequest.new |
|
32 | 32 | @response = ActionController::TestResponse.new |
|
33 | 33 | @request.session[:user_id] = nil |
|
34 | 34 | Setting.default_language = 'en' |
|
35 | 35 | end |
|
36 | 36 | |
|
37 | 37 | def test_index |
|
38 | 38 | get :index |
|
39 | 39 | assert_response :success |
|
40 | 40 | assert_template 'index' |
|
41 | 41 | assert_not_nil assigns(:projects) |
|
42 | 42 | |
|
43 | 43 | assert_tag :ul, :child => {:tag => 'li', |
|
44 | 44 | :descendant => {:tag => 'a', :content => 'eCookbook'}, |
|
45 | 45 | :child => { :tag => 'ul', |
|
46 | 46 | :descendant => { :tag => 'a', |
|
47 | 47 | :content => 'Child of private child' |
|
48 | 48 | } |
|
49 | 49 | } |
|
50 | 50 | } |
|
51 | 51 | |
|
52 | 52 | assert_no_tag :a, :content => /Private child of eCookbook/ |
|
53 | 53 | end |
|
54 | 54 | |
|
55 | 55 | def test_index_atom |
|
56 | 56 | get :index, :format => 'atom' |
|
57 | 57 | assert_response :success |
|
58 | 58 | assert_template 'common/feed.atom' |
|
59 | 59 | assert_select 'feed>title', :text => 'Redmine: Latest projects' |
|
60 | 60 | assert_select 'feed>entry', :count => Project.count(:conditions => Project.visible_condition(User.current)) |
|
61 | 61 | end |
|
62 | 62 | |
|
63 | 63 | context "#index" do |
|
64 | 64 | context "by non-admin user with view_time_entries permission" do |
|
65 | 65 | setup do |
|
66 | 66 | @request.session[:user_id] = 3 |
|
67 | 67 | end |
|
68 | 68 | should "show overall spent time link" do |
|
69 | 69 | get :index |
|
70 | 70 | assert_template 'index' |
|
71 | 71 | assert_tag :a, :attributes => {:href => '/time_entries'} |
|
72 | 72 | end |
|
73 | 73 | end |
|
74 | 74 | |
|
75 | 75 | context "by non-admin user without view_time_entries permission" do |
|
76 | 76 | setup do |
|
77 | 77 | Role.find(2).remove_permission! :view_time_entries |
|
78 | 78 | Role.non_member.remove_permission! :view_time_entries |
|
79 | 79 | Role.anonymous.remove_permission! :view_time_entries |
|
80 | 80 | @request.session[:user_id] = 3 |
|
81 | 81 | end |
|
82 | 82 | should "not show overall spent time link" do |
|
83 | 83 | get :index |
|
84 | 84 | assert_template 'index' |
|
85 | 85 | assert_no_tag :a, :attributes => {:href => '/time_entries'} |
|
86 | 86 | end |
|
87 | 87 | end |
|
88 | 88 | end |
|
89 | 89 | |
|
90 | 90 | context "#new" do |
|
91 | 91 | context "by admin user" do |
|
92 | 92 | setup do |
|
93 | 93 | @request.session[:user_id] = 1 |
|
94 | 94 | end |
|
95 | 95 | |
|
96 | 96 | should "accept get" do |
|
97 | 97 | get :new |
|
98 | 98 | assert_response :success |
|
99 | 99 | assert_template 'new' |
|
100 | 100 | end |
|
101 | 101 | |
|
102 | 102 | end |
|
103 | 103 | |
|
104 | 104 | context "by non-admin user with add_project permission" do |
|
105 | 105 | setup do |
|
106 | 106 | Role.non_member.add_permission! :add_project |
|
107 | 107 | @request.session[:user_id] = 9 |
|
108 | 108 | end |
|
109 | 109 | |
|
110 | 110 | should "accept get" do |
|
111 | 111 | get :new |
|
112 | 112 | assert_response :success |
|
113 | 113 | assert_template 'new' |
|
114 | 114 | assert_no_tag :select, :attributes => {:name => 'project[parent_id]'} |
|
115 | 115 | end |
|
116 | 116 | end |
|
117 | 117 | |
|
118 | 118 | context "by non-admin user with add_subprojects permission" do |
|
119 | 119 | setup do |
|
120 | 120 | Role.find(1).remove_permission! :add_project |
|
121 | 121 | Role.find(1).add_permission! :add_subprojects |
|
122 | 122 | @request.session[:user_id] = 2 |
|
123 | 123 | end |
|
124 | 124 | |
|
125 | 125 | should "accept get" do |
|
126 | 126 | get :new, :parent_id => 'ecookbook' |
|
127 | 127 | assert_response :success |
|
128 | 128 | assert_template 'new' |
|
129 | 129 | # parent project selected |
|
130 | 130 | assert_tag :select, :attributes => {:name => 'project[parent_id]'}, |
|
131 | 131 | :child => {:tag => 'option', :attributes => {:value => '1', :selected => 'selected'}} |
|
132 | 132 | # no empty value |
|
133 | 133 | assert_no_tag :select, :attributes => {:name => 'project[parent_id]'}, |
|
134 | 134 | :child => {:tag => 'option', :attributes => {:value => ''}} |
|
135 | 135 | end |
|
136 | 136 | end |
|
137 | 137 | |
|
138 | 138 | end |
|
139 | 139 | |
|
140 | 140 | context "POST :create" do |
|
141 | 141 | context "by admin user" do |
|
142 | 142 | setup do |
|
143 | 143 | @request.session[:user_id] = 1 |
|
144 | 144 | end |
|
145 | 145 | |
|
146 | 146 | should "create a new project" do |
|
147 | 147 | post :create, |
|
148 | 148 | :project => { |
|
149 | 149 | :name => "blog", |
|
150 | 150 | :description => "weblog", |
|
151 | 151 | :homepage => 'http://weblog', |
|
152 | 152 | :identifier => "blog", |
|
153 | 153 | :is_public => 1, |
|
154 | 154 | :custom_field_values => { '3' => 'Beta' }, |
|
155 | 155 | :tracker_ids => ['1', '3'], |
|
156 | 156 | # an issue custom field that is not for all project |
|
157 | 157 | :issue_custom_field_ids => ['9'], |
|
158 | 158 | :enabled_module_names => ['issue_tracking', 'news', 'repository'] |
|
159 | 159 | } |
|
160 | 160 | assert_redirected_to '/projects/blog/settings' |
|
161 | 161 | |
|
162 | 162 | project = Project.find_by_name('blog') |
|
163 | 163 | assert_kind_of Project, project |
|
164 | 164 | assert project.active? |
|
165 | 165 | assert_equal 'weblog', project.description |
|
166 | 166 | assert_equal 'http://weblog', project.homepage |
|
167 | 167 | assert_equal true, project.is_public? |
|
168 | 168 | assert_nil project.parent |
|
169 | 169 | assert_equal 'Beta', project.custom_value_for(3).value |
|
170 | 170 | assert_equal [1, 3], project.trackers.map(&:id).sort |
|
171 | 171 | assert_equal ['issue_tracking', 'news', 'repository'], project.enabled_module_names.sort |
|
172 | 172 | assert project.issue_custom_fields.include?(IssueCustomField.find(9)) |
|
173 | 173 | end |
|
174 | 174 | |
|
175 | 175 | should "create a new subproject" do |
|
176 | 176 | post :create, :project => { :name => "blog", |
|
177 | 177 | :description => "weblog", |
|
178 | 178 | :identifier => "blog", |
|
179 | 179 | :is_public => 1, |
|
180 | 180 | :custom_field_values => { '3' => 'Beta' }, |
|
181 | 181 | :parent_id => 1 |
|
182 | 182 | } |
|
183 | 183 | assert_redirected_to '/projects/blog/settings' |
|
184 | 184 | |
|
185 | 185 | project = Project.find_by_name('blog') |
|
186 | 186 | assert_kind_of Project, project |
|
187 | 187 | assert_equal Project.find(1), project.parent |
|
188 | 188 | end |
|
189 | 189 | |
|
190 | 190 | should "continue" do |
|
191 | 191 | assert_difference 'Project.count' do |
|
192 | 192 | post :create, :project => {:name => "blog", :identifier => "blog"}, :continue => 'Create and continue' |
|
193 | 193 | end |
|
194 | 194 | assert_redirected_to '/projects/new?' |
|
195 | 195 | end |
|
196 | 196 | end |
|
197 | 197 | |
|
198 | 198 | context "by non-admin user with add_project permission" do |
|
199 | 199 | setup do |
|
200 | 200 | Role.non_member.add_permission! :add_project |
|
201 | 201 | @request.session[:user_id] = 9 |
|
202 | 202 | end |
|
203 | 203 | |
|
204 | 204 | should "accept create a Project" do |
|
205 | 205 | post :create, :project => { :name => "blog", |
|
206 | 206 | :description => "weblog", |
|
207 | 207 | :identifier => "blog", |
|
208 | 208 | :is_public => 1, |
|
209 | 209 | :custom_field_values => { '3' => 'Beta' }, |
|
210 | 210 | :tracker_ids => ['1', '3'], |
|
211 | 211 | :enabled_module_names => ['issue_tracking', 'news', 'repository'] |
|
212 | 212 | } |
|
213 | 213 | |
|
214 | 214 | assert_redirected_to '/projects/blog/settings' |
|
215 | 215 | |
|
216 | 216 | project = Project.find_by_name('blog') |
|
217 | 217 | assert_kind_of Project, project |
|
218 | 218 | assert_equal 'weblog', project.description |
|
219 | 219 | assert_equal true, project.is_public? |
|
220 | 220 | assert_equal [1, 3], project.trackers.map(&:id).sort |
|
221 | 221 | assert_equal ['issue_tracking', 'news', 'repository'], project.enabled_module_names.sort |
|
222 | 222 | |
|
223 | 223 | # User should be added as a project member |
|
224 | 224 | assert User.find(9).member_of?(project) |
|
225 | 225 | assert_equal 1, project.members.size |
|
226 | 226 | end |
|
227 | 227 | |
|
228 | 228 | should "fail with parent_id" do |
|
229 | 229 | assert_no_difference 'Project.count' do |
|
230 | 230 | post :create, :project => { :name => "blog", |
|
231 | 231 | :description => "weblog", |
|
232 | 232 | :identifier => "blog", |
|
233 | 233 | :is_public => 1, |
|
234 | 234 | :custom_field_values => { '3' => 'Beta' }, |
|
235 | 235 | :parent_id => 1 |
|
236 | 236 | } |
|
237 | 237 | end |
|
238 | 238 | assert_response :success |
|
239 | 239 | project = assigns(:project) |
|
240 | 240 | assert_kind_of Project, project |
|
241 | 241 | assert_not_nil project.errors[:parent_id] |
|
242 | 242 | end |
|
243 | 243 | end |
|
244 | 244 | |
|
245 | 245 | context "by non-admin user with add_subprojects permission" do |
|
246 | 246 | setup do |
|
247 | 247 | Role.find(1).remove_permission! :add_project |
|
248 | 248 | Role.find(1).add_permission! :add_subprojects |
|
249 | 249 | @request.session[:user_id] = 2 |
|
250 | 250 | end |
|
251 | 251 | |
|
252 | 252 | should "create a project with a parent_id" do |
|
253 | 253 | post :create, :project => { :name => "blog", |
|
254 | 254 | :description => "weblog", |
|
255 | 255 | :identifier => "blog", |
|
256 | 256 | :is_public => 1, |
|
257 | 257 | :custom_field_values => { '3' => 'Beta' }, |
|
258 | 258 | :parent_id => 1 |
|
259 | 259 | } |
|
260 | 260 | assert_redirected_to '/projects/blog/settings' |
|
261 | 261 | project = Project.find_by_name('blog') |
|
262 | 262 | end |
|
263 | 263 | |
|
264 | 264 | should "fail without parent_id" do |
|
265 | 265 | assert_no_difference 'Project.count' do |
|
266 | 266 | post :create, :project => { :name => "blog", |
|
267 | 267 | :description => "weblog", |
|
268 | 268 | :identifier => "blog", |
|
269 | 269 | :is_public => 1, |
|
270 | 270 | :custom_field_values => { '3' => 'Beta' } |
|
271 | 271 | } |
|
272 | 272 | end |
|
273 | 273 | assert_response :success |
|
274 | 274 | project = assigns(:project) |
|
275 | 275 | assert_kind_of Project, project |
|
276 | 276 | assert_not_nil project.errors[:parent_id] |
|
277 | 277 | end |
|
278 | 278 | |
|
279 | 279 | should "fail with unauthorized parent_id" do |
|
280 | 280 | assert !User.find(2).member_of?(Project.find(6)) |
|
281 | 281 | assert_no_difference 'Project.count' do |
|
282 | 282 | post :create, :project => { :name => "blog", |
|
283 | 283 | :description => "weblog", |
|
284 | 284 | :identifier => "blog", |
|
285 | 285 | :is_public => 1, |
|
286 | 286 | :custom_field_values => { '3' => 'Beta' }, |
|
287 | 287 | :parent_id => 6 |
|
288 | 288 | } |
|
289 | 289 | end |
|
290 | 290 | assert_response :success |
|
291 | 291 | project = assigns(:project) |
|
292 | 292 | assert_kind_of Project, project |
|
293 | 293 | assert_not_nil project.errors[:parent_id] |
|
294 | 294 | end |
|
295 | 295 | end |
|
296 | 296 | end |
|
297 | 297 | |
|
298 | 298 | def test_create_should_preserve_modules_on_validation_failure |
|
299 | 299 | with_settings :default_projects_modules => ['issue_tracking', 'repository'] do |
|
300 | 300 | @request.session[:user_id] = 1 |
|
301 | 301 | assert_no_difference 'Project.count' do |
|
302 | 302 | post :create, :project => { |
|
303 | 303 | :name => "blog", |
|
304 | 304 | :identifier => "", |
|
305 | 305 | :enabled_module_names => %w(issue_tracking news) |
|
306 | 306 | } |
|
307 | 307 | end |
|
308 | 308 | assert_response :success |
|
309 | 309 | project = assigns(:project) |
|
310 | 310 | assert_equal %w(issue_tracking news), project.enabled_module_names.sort |
|
311 | 311 | end |
|
312 | 312 | end |
|
313 | 313 | |
|
314 | 314 | def test_create_should_not_accept_get |
|
315 | 315 | @request.session[:user_id] = 1 |
|
316 | 316 | get :create |
|
317 | 317 | assert_response :method_not_allowed |
|
318 | 318 | end |
|
319 | 319 | |
|
320 | 320 | def test_show_by_id |
|
321 | 321 | get :show, :id => 1 |
|
322 | 322 | assert_response :success |
|
323 | 323 | assert_template 'show' |
|
324 | 324 | assert_not_nil assigns(:project) |
|
325 | 325 | end |
|
326 | 326 | |
|
327 | 327 | def test_show_by_identifier |
|
328 | 328 | get :show, :id => 'ecookbook' |
|
329 | 329 | assert_response :success |
|
330 | 330 | assert_template 'show' |
|
331 | 331 | assert_not_nil assigns(:project) |
|
332 | 332 | assert_equal Project.find_by_identifier('ecookbook'), assigns(:project) |
|
333 | 333 | |
|
334 | 334 | assert_tag 'li', :content => /Development status/ |
|
335 | 335 | end |
|
336 | 336 | |
|
337 | 337 | def test_show_should_not_display_hidden_custom_fields |
|
338 | 338 | ProjectCustomField.find_by_name('Development status').update_attribute :visible, false |
|
339 | 339 | get :show, :id => 'ecookbook' |
|
340 | 340 | assert_response :success |
|
341 | 341 | assert_template 'show' |
|
342 | 342 | assert_not_nil assigns(:project) |
|
343 | 343 | |
|
344 | 344 | assert_no_tag 'li', :content => /Development status/ |
|
345 | 345 | end |
|
346 | 346 | |
|
347 | 347 | def test_show_should_not_fail_when_custom_values_are_nil |
|
348 | 348 | project = Project.find_by_identifier('ecookbook') |
|
349 | 349 | project.custom_values.first.update_attribute(:value, nil) |
|
350 | 350 | get :show, :id => 'ecookbook' |
|
351 | 351 | assert_response :success |
|
352 | 352 | assert_template 'show' |
|
353 | 353 | assert_not_nil assigns(:project) |
|
354 | 354 | assert_equal Project.find_by_identifier('ecookbook'), assigns(:project) |
|
355 | 355 | end |
|
356 | 356 | |
|
357 | 357 | def show_archived_project_should_be_denied |
|
358 | 358 | project = Project.find_by_identifier('ecookbook') |
|
359 | 359 | project.archive! |
|
360 | 360 | |
|
361 | 361 | get :show, :id => 'ecookbook' |
|
362 | 362 | assert_response 403 |
|
363 | 363 | assert_nil assigns(:project) |
|
364 | 364 | assert_tag :tag => 'p', :content => /archived/ |
|
365 | 365 | end |
|
366 | 366 | |
|
367 | 367 | def test_private_subprojects_hidden |
|
368 | 368 | get :show, :id => 'ecookbook' |
|
369 | 369 | assert_response :success |
|
370 | 370 | assert_template 'show' |
|
371 | 371 | assert_no_tag :tag => 'a', :content => /Private child/ |
|
372 | 372 | end |
|
373 | 373 | |
|
374 | 374 | def test_private_subprojects_visible |
|
375 | 375 | @request.session[:user_id] = 2 # manager who is a member of the private subproject |
|
376 | 376 | get :show, :id => 'ecookbook' |
|
377 | 377 | assert_response :success |
|
378 | 378 | assert_template 'show' |
|
379 | 379 | assert_tag :tag => 'a', :content => /Private child/ |
|
380 | 380 | end |
|
381 | 381 | |
|
382 | 382 | def test_settings |
|
383 | 383 | @request.session[:user_id] = 2 # manager |
|
384 | 384 | get :settings, :id => 1 |
|
385 | 385 | assert_response :success |
|
386 | 386 | assert_template 'settings' |
|
387 | 387 | end |
|
388 | 388 | |
|
389 | 389 | def test_update |
|
390 | 390 | @request.session[:user_id] = 2 # manager |
|
391 | 391 | post :update, :id => 1, :project => {:name => 'Test changed name', |
|
392 | 392 | :issue_custom_field_ids => ['']} |
|
393 | 393 | assert_redirected_to '/projects/ecookbook/settings' |
|
394 | 394 | project = Project.find(1) |
|
395 | 395 | assert_equal 'Test changed name', project.name |
|
396 | 396 | end |
|
397 | 397 | |
|
398 | 398 | def test_modules |
|
399 | 399 | @request.session[:user_id] = 2 |
|
400 | 400 | Project.find(1).enabled_module_names = ['issue_tracking', 'news'] |
|
401 | 401 | |
|
402 | 402 | post :modules, :id => 1, :enabled_module_names => ['issue_tracking', 'repository', 'documents'] |
|
403 | 403 | assert_redirected_to '/projects/ecookbook/settings/modules' |
|
404 | 404 | assert_equal ['documents', 'issue_tracking', 'repository'], Project.find(1).enabled_module_names.sort |
|
405 | 405 | end |
|
406 | 406 | |
|
407 | 407 | def test_modules_should_not_allow_get |
|
408 | 408 | @request.session[:user_id] = 1 |
|
409 | 409 | get :modules, :id => 1 |
|
410 | 410 | assert_response :method_not_allowed |
|
411 | 411 | end |
|
412 | 412 | |
|
413 | 413 | def test_destroy_without_confirmation |
|
414 | 414 | @request.session[:user_id] = 1 # admin |
|
415 | 415 | delete :destroy, :id => 1 |
|
416 | 416 | assert_response :success |
|
417 | 417 | assert_template 'destroy' |
|
418 | 418 | assert_not_nil Project.find_by_id(1) |
|
419 | 419 | end |
|
420 | 420 | |
|
421 | 421 | def test_destroy |
|
422 | 422 | @request.session[:user_id] = 1 # admin |
|
423 | 423 | delete :destroy, :id => 1, :confirm => 1 |
|
424 | 424 | assert_redirected_to '/admin/projects' |
|
425 | 425 | assert_nil Project.find_by_id(1) |
|
426 | 426 | end |
|
427 | 427 | |
|
428 | 428 | def test_archive |
|
429 | 429 | @request.session[:user_id] = 1 # admin |
|
430 | 430 | post :archive, :id => 1 |
|
431 | 431 | assert_redirected_to '/admin/projects' |
|
432 | 432 | assert !Project.find(1).active? |
|
433 | 433 | end |
|
434 | 434 | |
|
435 | 435 | def test_unarchive |
|
436 | 436 | @request.session[:user_id] = 1 # admin |
|
437 | 437 | Project.find(1).archive |
|
438 | 438 | post :unarchive, :id => 1 |
|
439 | 439 | assert_redirected_to '/admin/projects' |
|
440 | 440 | assert Project.find(1).active? |
|
441 | 441 | end |
|
442 | 442 | |
|
443 | 443 | def test_project_breadcrumbs_should_be_limited_to_3_ancestors |
|
444 | 444 | CustomField.delete_all |
|
445 | 445 | parent = nil |
|
446 | 446 | 6.times do |i| |
|
447 | 447 | p = Project.create!(:name => "Breadcrumbs #{i}", :identifier => "breadcrumbs-#{i}") |
|
448 | 448 | p.set_parent!(parent) |
|
449 | 449 | get :show, :id => p |
|
450 | 450 | assert_tag :h1, :parent => { :attributes => {:id => 'header'}}, |
|
451 | 451 | :children => { :count => [i, 3].min, |
|
452 | 452 | :only => { :tag => 'a' } } |
|
453 | 453 | |
|
454 | 454 | parent = p |
|
455 | 455 | end |
|
456 | 456 | end |
|
457 | 457 | |
|
458 | 458 | def test_get_copy |
|
459 | 459 | @request.session[:user_id] = 1 # admin |
|
460 | 460 | get :copy, :id => 1 |
|
461 | 461 | assert_response :success |
|
462 | 462 | assert_template 'copy' |
|
463 | 463 | assert assigns(:project) |
|
464 | 464 | assert_equal Project.find(1).description, assigns(:project).description |
|
465 | 465 | assert_nil assigns(:project).id |
|
466 | 466 | |
|
467 | 467 | assert_tag :tag => 'input', |
|
468 | 468 | :attributes => {:name => 'project[enabled_module_names][]', :value => 'issue_tracking'} |
|
469 | 469 | end |
|
470 | 470 | |
|
471 | def test_get_copy_without_project | |
|
472 | @request.session[:user_id] = 1 # admin | |
|
473 | get :copy | |
|
474 | assert_response :redirect | |
|
475 | assert_redirected_to :controller => 'admin', :action => 'projects' | |
|
476 | end | |
|
477 | ||
|
478 | 471 | def test_post_copy_should_copy_requested_items |
|
479 | 472 | @request.session[:user_id] = 1 # admin |
|
480 | 473 | CustomField.delete_all |
|
481 | 474 | |
|
482 | 475 | assert_difference 'Project.count' do |
|
483 | 476 | post :copy, :id => 1, |
|
484 | 477 | :project => { |
|
485 | 478 | :name => 'Copy', |
|
486 | 479 | :identifier => 'unique-copy', |
|
487 | 480 | :tracker_ids => ['1', '2', '3', ''], |
|
488 | 481 | :enabled_module_names => %w(issue_tracking time_tracking) |
|
489 | 482 | }, |
|
490 | 483 | :only => %w(issues versions) |
|
491 | 484 | end |
|
492 | 485 | project = Project.find('unique-copy') |
|
493 | 486 | source = Project.find(1) |
|
494 | 487 | assert_equal %w(issue_tracking time_tracking), project.enabled_module_names.sort |
|
495 | 488 | |
|
496 | 489 | assert_equal source.versions.count, project.versions.count, "All versions were not copied" |
|
497 | 490 | # issues assigned to a closed version won't be copied |
|
498 | 491 | assert_equal source.issues.select {|i| i.fixed_version.nil? || i.fixed_version.open?}.size, |
|
499 | 492 | project.issues.count, "All issues were not copied" |
|
500 | 493 | assert_equal 0, project.members.count |
|
501 | 494 | end |
|
502 | 495 | |
|
503 | 496 | def test_post_copy_should_redirect_to_settings_when_successful |
|
504 | 497 | @request.session[:user_id] = 1 # admin |
|
505 | 498 | post :copy, :id => 1, :project => {:name => 'Copy', :identifier => 'unique-copy'} |
|
506 | 499 | assert_response :redirect |
|
507 | 500 | assert_redirected_to :controller => 'projects', :action => 'settings', :id => 'unique-copy' |
|
508 | 501 | end |
|
509 | 502 | |
|
510 | 503 | def test_jump_should_redirect_to_active_tab |
|
511 | 504 | get :show, :id => 1, :jump => 'issues' |
|
512 | 505 | assert_redirected_to '/projects/ecookbook/issues' |
|
513 | 506 | end |
|
514 | 507 | |
|
515 | 508 | def test_jump_should_not_redirect_to_inactive_tab |
|
516 | 509 | get :show, :id => 3, :jump => 'documents' |
|
517 | 510 | assert_response :success |
|
518 | 511 | assert_template 'show' |
|
519 | 512 | end |
|
520 | 513 | |
|
521 | 514 | def test_jump_should_not_redirect_to_unknown_tab |
|
522 | 515 | get :show, :id => 3, :jump => 'foobar' |
|
523 | 516 | assert_response :success |
|
524 | 517 | assert_template 'show' |
|
525 | 518 | end |
|
526 | 519 | |
|
527 | 520 | # A hook that is manually registered later |
|
528 | 521 | class ProjectBasedTemplate < Redmine::Hook::ViewListener |
|
529 | 522 | def view_layouts_base_html_head(context) |
|
530 | 523 | # Adds a project stylesheet |
|
531 | 524 | stylesheet_link_tag(context[:project].identifier) if context[:project] |
|
532 | 525 | end |
|
533 | 526 | end |
|
534 | 527 | # Don't use this hook now |
|
535 | 528 | Redmine::Hook.clear_listeners |
|
536 | 529 | |
|
537 | 530 | def test_hook_response |
|
538 | 531 | Redmine::Hook.add_listener(ProjectBasedTemplate) |
|
539 | 532 | get :show, :id => 1 |
|
540 | 533 | assert_tag :tag => 'link', :attributes => {:href => '/stylesheets/ecookbook.css'}, |
|
541 | 534 | :parent => {:tag => 'head'} |
|
542 | 535 | |
|
543 | 536 | Redmine::Hook.clear_listeners |
|
544 | 537 | end |
|
545 | 538 | end |
@@ -1,62 +1,62 | |||
|
1 | 1 | # Redmine - project management software |
|
2 | 2 | # Copyright (C) 2006-2011 Jean-Philippe Lang |
|
3 | 3 | # |
|
4 | 4 | # This program is free software; you can redistribute it and/or |
|
5 | 5 | # modify it under the terms of the GNU General Public License |
|
6 | 6 | # as published by the Free Software Foundation; either version 2 |
|
7 | 7 | # of the License, or (at your option) any later version. |
|
8 | 8 | # |
|
9 | 9 | # This program is distributed in the hope that it will be useful, |
|
10 | 10 | # but WITHOUT ANY WARRANTY; without even the implied warranty of |
|
11 | 11 | # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
|
12 | 12 | # GNU General Public License for more details. |
|
13 | 13 | # |
|
14 | 14 | # You should have received a copy of the GNU General Public License |
|
15 | 15 | # along with this program; if not, write to the Free Software |
|
16 | 16 | # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. |
|
17 | 17 | |
|
18 | 18 | require File.expand_path('../../test_helper', __FILE__) |
|
19 | 19 | |
|
20 | 20 | class AdminTest < ActionController::IntegrationTest |
|
21 | 21 | fixtures :projects, :trackers, :issue_statuses, :issues, |
|
22 | 22 | :enumerations, :users, :issue_categories, |
|
23 | 23 | :projects_trackers, |
|
24 | 24 | :roles, |
|
25 | 25 | :member_roles, |
|
26 | 26 | :members, |
|
27 | 27 | :enabled_modules, |
|
28 | 28 | :workflows |
|
29 | 29 | |
|
30 | 30 | def test_add_user |
|
31 | 31 | log_user("admin", "admin") |
|
32 | 32 | get "/users/new" |
|
33 | 33 | assert_response :success |
|
34 | 34 | assert_template "users/new" |
|
35 |
post "/users |
|
|
35 | post "/users", | |
|
36 | 36 | :user => { :login => "psmith", :firstname => "Paul", |
|
37 | 37 | :lastname => "Smith", :mail => "psmith@somenet.foo", |
|
38 | 38 | :language => "en", :password => "psmith09", |
|
39 | 39 | :password_confirmation => "psmith09" } |
|
40 | 40 | |
|
41 | 41 | user = User.find_by_login("psmith") |
|
42 | 42 | assert_kind_of User, user |
|
43 | 43 | assert_redirected_to "/users/#{ user.id }/edit" |
|
44 | 44 | |
|
45 | 45 | logged_user = User.try_to_login("psmith", "psmith09") |
|
46 | 46 | assert_kind_of User, logged_user |
|
47 | 47 | assert_equal "Paul", logged_user.firstname |
|
48 | 48 | |
|
49 | 49 | put "users/#{user.id}", :id => user.id, :user => { :status => User::STATUS_LOCKED } |
|
50 | 50 | assert_redirected_to "/users/#{ user.id }/edit" |
|
51 | 51 | locked_user = User.try_to_login("psmith", "psmith09") |
|
52 | 52 | assert_equal nil, locked_user |
|
53 | 53 | end |
|
54 | 54 | |
|
55 | 55 | test "Add a user as an anonymous user should fail" do |
|
56 |
post '/users |
|
|
56 | post '/users', | |
|
57 | 57 | :user => { :login => 'psmith', :firstname => 'Paul'}, |
|
58 | 58 | :password => "psmith09", :password_confirmation => "psmith09" |
|
59 | 59 | assert_response :redirect |
|
60 | 60 | assert_redirected_to "/login?back_url=http%3A%2F%2Fwww.example.com%2Fusers" |
|
61 | 61 | end |
|
62 | 62 | end |
@@ -1,44 +1,44 | |||
|
1 | 1 | # Redmine - project management software |
|
2 | 2 | # Copyright (C) 2006-2011 Jean-Philippe Lang |
|
3 | 3 | # |
|
4 | 4 | # This program is free software; you can redistribute it and/or |
|
5 | 5 | # modify it under the terms of the GNU General Public License |
|
6 | 6 | # as published by the Free Software Foundation; either version 2 |
|
7 | 7 | # of the License, or (at your option) any later version. |
|
8 | 8 | # |
|
9 | 9 | # This program is distributed in the hope that it will be useful, |
|
10 | 10 | # but WITHOUT ANY WARRANTY; without even the implied warranty of |
|
11 | 11 | # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
|
12 | 12 | # GNU General Public License for more details. |
|
13 | 13 | # |
|
14 | 14 | # You should have received a copy of the GNU General Public License |
|
15 | 15 | # along with this program; if not, write to the Free Software |
|
16 | 16 | # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. |
|
17 | 17 | |
|
18 | 18 | require File.expand_path('../../test_helper', __FILE__) |
|
19 | 19 | |
|
20 | 20 | class ProjectsTest < ActionController::IntegrationTest |
|
21 | 21 | fixtures :projects, :users, :members |
|
22 | 22 | |
|
23 | 23 | def test_archive_project |
|
24 | 24 | subproject = Project.find(1).children.first |
|
25 | 25 | log_user("admin", "admin") |
|
26 | 26 | get "admin/projects" |
|
27 | 27 | assert_response :success |
|
28 | 28 | assert_template "admin/projects" |
|
29 |
post "projects/archive" |
|
|
29 | post "projects/1/archive" | |
|
30 | 30 | assert_redirected_to "/admin/projects" |
|
31 | 31 | assert !Project.find(1).active? |
|
32 | 32 | |
|
33 | 33 | get 'projects/1' |
|
34 | 34 | assert_response 403 |
|
35 | 35 | get "projects/#{subproject.id}" |
|
36 | 36 | assert_response 403 |
|
37 | 37 | |
|
38 |
post "projects/unarchive" |
|
|
38 | post "projects/1/unarchive" | |
|
39 | 39 | assert_redirected_to "/admin/projects" |
|
40 | 40 | assert Project.find(1).active? |
|
41 | 41 | get "projects/1" |
|
42 | 42 | assert_response :success |
|
43 | 43 | end |
|
44 | 44 | end |
General Comments 0
You need to be logged in to leave comments.
Login now