Commit e327934dec6a8c16356ea8b89231a0f8c3220e1b
1 parent
55f802bd
Exists in
api_tasks
Removing specific resources of task
Showing
2 changed files
with
140 additions
and
84 deletions
Show diff stats
lib/noosfero/api/helpers.rb
... | ... | @@ -45,7 +45,10 @@ module Noosfero |
45 | 45 | end |
46 | 46 | end |
47 | 47 | |
48 | - ARTICLE_TYPES = Article.descendants.map{|a| a.to_s} | |
48 | + Rails.application.eager_load! | |
49 | + ARTICLE_TYPES = ['Article'] + Article.descendants.map{|a| a.to_s} | |
50 | + TASK_TYPES = ['Task'] + Task.descendants.map{|a| a.to_s} | |
51 | + | |
49 | 52 | |
50 | 53 | def find_article(articles, id) |
51 | 54 | article = articles.find(id) |
... | ... | @@ -85,6 +88,34 @@ module Noosfero |
85 | 88 | task.display_to?(current_user.person) ? task : forbidden! |
86 | 89 | end |
87 | 90 | |
91 | + def post_task(asset, params) | |
92 | + return forbidden! unless current_person.has_permission?(:perform_task, asset) | |
93 | + | |
94 | + klass_type= params[:content_type].nil? ? 'Task' : params[:content_type] | |
95 | + return forbidden! unless TASK_TYPES.include?(klass_type) | |
96 | + | |
97 | + task = klass_type.constantize.new(params[:task]) | |
98 | + task.requestor_id = current_person.id | |
99 | + task.target_id = asset.id | |
100 | + task.target_type = 'Profile' | |
101 | + | |
102 | + if !task.save | |
103 | + render_api_errors!(task.errors.full_messages) | |
104 | + end | |
105 | + present task, :with => Entities::Task, :fields => params[:fields] | |
106 | + end | |
107 | + | |
108 | + def present_task(asset) | |
109 | + task = find_task(asset.tasks, params[:id]) | |
110 | + present task, :with => Entities::Task, :fields => params[:fields] | |
111 | + end | |
112 | + | |
113 | + def present_tasks(asset) | |
114 | + tasks = select_filtered_collection_of(asset, 'tasks', params) | |
115 | + tasks = tasks.select {|t| t.display_to?(current_user.person)} | |
116 | + present tasks, :with => Entities::Task, :fields => params[:fields] | |
117 | + end | |
118 | + | |
88 | 119 | def make_conditions_with_parameter(params = {}) |
89 | 120 | parsed_params = parser_params(params) |
90 | 121 | conditions = {} | ... | ... |
lib/noosfero/api/v1/tasks.rb
... | ... | @@ -32,6 +32,31 @@ module Noosfero |
32 | 32 | |
33 | 33 | end |
34 | 34 | |
35 | + kinds = %w[community person enterprise] | |
36 | + kinds.each do |kind| | |
37 | + resource kind.pluralize.to_sym do | |
38 | + segment "/:#{kind}_id" do | |
39 | + resource :tasks do | |
40 | + get do | |
41 | + profile = environment.send(kind.pluralize).find(params["#{kind}_id"]) | |
42 | + present_tasks(profile) | |
43 | + end | |
44 | + | |
45 | + get ':id' do | |
46 | + profile = environment.send(kind.pluralize).find(params["#{kind}_id"]) | |
47 | + present_task(profile) | |
48 | + end | |
49 | + | |
50 | + post do | |
51 | + profile = environment.send(kind.pluralize).find(params["#{kind}_id"]) | |
52 | + post_task(profile, params) | |
53 | + end | |
54 | + end | |
55 | + end | |
56 | + end | |
57 | + end | |
58 | + | |
59 | + | |
35 | 60 | resource :communities do |
36 | 61 | segment '/:community_id' do |
37 | 62 | resource :tasks do |
... | ... | @@ -73,89 +98,89 @@ module Noosfero |
73 | 98 | |
74 | 99 | end |
75 | 100 | |
76 | - resource :people do | |
77 | - segment '/:person_id' do | |
78 | - resource :tasks do | |
79 | - get do | |
80 | -# person = environment.people.find(params[:person_id]) | |
81 | -# articles = select_filtered_collection_of(person, 'articles', params) | |
82 | -# articles = articles.display_filter(current_person, person) | |
83 | -tasks = Task.all | |
84 | - present tasks, :with => Entities::Task, :fields => params[:fields] | |
85 | - end | |
86 | - | |
87 | - get ':id' do | |
88 | -# person = environment.people.find(params[:person_id]) | |
89 | -# article = find_article(person.articles, params[:id]) | |
90 | -task = Task.first | |
91 | - present task, :with => Entities::Task, :fields => params[:fields] | |
92 | - end | |
93 | - | |
94 | - post do | |
95 | -# person = environment.people.find(params[:person_id]) | |
96 | -# return forbidden! unless current_person.can_post_content?(person) | |
97 | -# | |
98 | -# klass_type= params[:content_type].nil? ? 'TinyMceArticle' : params[:content_type] | |
99 | -# return forbidden! unless ARTICLE_TYPES.include?(klass_type) | |
100 | -# | |
101 | -# article = klass_type.constantize.new(params[:article]) | |
102 | -# article.last_changed_by = current_person | |
103 | -# article.created_by= current_person | |
104 | -# article.profile = person | |
105 | -# | |
106 | -# if !article.save | |
107 | -# render_api_errors!(article.errors.full_messages) | |
108 | -# end | |
109 | -task = Task.first | |
110 | - present task, :with => Entities::Task, :fields => params[:fields] | |
111 | - end | |
112 | - | |
113 | - end | |
114 | - end | |
115 | - | |
116 | - end | |
117 | - | |
118 | - resource :enterprises do | |
119 | - segment '/:enterprise_id' do | |
120 | - resource :tasks do | |
121 | - get do | |
122 | -# enterprise = environment.enterprises.find(params[:enterprise_id]) | |
123 | -# articles = select_filtered_collection_of(enterprise, 'articles', params) | |
124 | -# articles = articles.display_filter(current_person, enterprise) | |
125 | -tasks = Task.all | |
126 | - present tasks, :with => Entities::Task, :fields => params[:fields] | |
127 | - end | |
128 | - | |
129 | - get ':id' do | |
130 | -# enterprise = environment.enterprises.find(params[:enterprise_id]) | |
131 | -# article = find_article(enterprise.articles, params[:id]) | |
132 | -task = Task.first | |
133 | - present task, :with => Entities::Task, :fields => params[:fields] | |
134 | - end | |
135 | - | |
136 | - post do | |
137 | -# enterprise = environment.enterprises.find(params[:enterprise_id]) | |
138 | -# return forbidden! unless current_person.can_post_content?(enterprise) | |
139 | -# | |
140 | -# klass_type= params[:content_type].nil? ? 'TinyMceArticle' : params[:content_type] | |
141 | -# return forbidden! unless ARTICLE_TYPES.include?(klass_type) | |
142 | -# | |
143 | -# article = klass_type.constantize.new(params[:article]) | |
144 | -# article.last_changed_by = current_person | |
145 | -# article.created_by= current_person | |
146 | -# article.profile = enterprise | |
147 | -# | |
148 | -# if !article.save | |
149 | -# render_api_errors!(article.errors.full_messages) | |
150 | -# end | |
151 | -task = Task.first | |
152 | - present task, :with => Entities::Task, :fields => params[:fields] | |
153 | - end | |
154 | - | |
155 | - end | |
156 | - end | |
157 | - | |
158 | - end | |
101 | +# resource :people do | |
102 | +# segment '/:person_id' do | |
103 | +# resource :tasks do | |
104 | +# get do | |
105 | +# # person = environment.people.find(params[:person_id]) | |
106 | +# # articles = select_filtered_collection_of(person, 'articles', params) | |
107 | +# # articles = articles.display_filter(current_person, person) | |
108 | +# tasks = Task.all | |
109 | +# present tasks, :with => Entities::Task, :fields => params[:fields] | |
110 | +# end | |
111 | + | |
112 | +# get ':id' do | |
113 | +# # person = environment.people.find(params[:person_id]) | |
114 | +# # article = find_article(person.articles, params[:id]) | |
115 | +# task = Task.first | |
116 | +# present task, :with => Entities::Task, :fields => params[:fields] | |
117 | +# end | |
118 | + | |
119 | +# post do | |
120 | +# # person = environment.people.find(params[:person_id]) | |
121 | +# # return forbidden! unless current_person.can_post_content?(person) | |
122 | +# # | |
123 | +# # klass_type= params[:content_type].nil? ? 'TinyMceArticle' : params[:content_type] | |
124 | +# # return forbidden! unless ARTICLE_TYPES.include?(klass_type) | |
125 | +# # | |
126 | +# # article = klass_type.constantize.new(params[:article]) | |
127 | +# # article.last_changed_by = current_person | |
128 | +# # article.created_by= current_person | |
129 | +# # article.profile = person | |
130 | +# # | |
131 | +# # if !article.save | |
132 | +# # render_api_errors!(article.errors.full_messages) | |
133 | +# # end | |
134 | +# task = Task.first | |
135 | +# present task, :with => Entities::Task, :fields => params[:fields] | |
136 | +# end | |
137 | + | |
138 | +# end | |
139 | +# end | |
140 | + | |
141 | +# end | |
142 | + | |
143 | +# resource :enterprises do | |
144 | +# segment '/:enterprise_id' do | |
145 | +# resource :tasks do | |
146 | +# get do | |
147 | +# # enterprise = environment.enterprises.find(params[:enterprise_id]) | |
148 | +# # articles = select_filtered_collection_of(enterprise, 'articles', params) | |
149 | +# # articles = articles.display_filter(current_person, enterprise) | |
150 | +# tasks = Task.all | |
151 | +# present tasks, :with => Entities::Task, :fields => params[:fields] | |
152 | +# end | |
153 | + | |
154 | +# get ':id' do | |
155 | +# # enterprise = environment.enterprises.find(params[:enterprise_id]) | |
156 | +# # article = find_article(enterprise.articles, params[:id]) | |
157 | +# task = Task.first | |
158 | +# present task, :with => Entities::Task, :fields => params[:fields] | |
159 | +# end | |
160 | + | |
161 | +# post do | |
162 | +# # enterprise = environment.enterprises.find(params[:enterprise_id]) | |
163 | +# # return forbidden! unless current_person.can_post_content?(enterprise) | |
164 | +# # | |
165 | +# # klass_type= params[:content_type].nil? ? 'TinyMceArticle' : params[:content_type] | |
166 | +# # return forbidden! unless ARTICLE_TYPES.include?(klass_type) | |
167 | +# # | |
168 | +# # article = klass_type.constantize.new(params[:article]) | |
169 | +# # article.last_changed_by = current_person | |
170 | +# # article.created_by= current_person | |
171 | +# # article.profile = enterprise | |
172 | +# # | |
173 | +# # if !article.save | |
174 | +# # render_api_errors!(article.errors.full_messages) | |
175 | +# # end | |
176 | +# task = Task.first | |
177 | +# present task, :with => Entities::Task, :fields => params[:fields] | |
178 | +# end | |
179 | + | |
180 | +# end | |
181 | +# end | |
182 | + | |
183 | +# end | |
159 | 184 | |
160 | 185 | |
161 | 186 | end | ... | ... |