##// END OF EJS Templates
pull request: show summary and creation date in table
Mads Kiilerich -
r3248:0dfe7eaf beta
parent child Browse files
Show More
@@ -1,228 +1,240 b''
1 <%inherit file="/base/base.html"/>
1 <%inherit file="/base/base.html"/>
2
2
3 <%def name="title()">
3 <%def name="title()">
4 ${c.repo_name} ${_('Pull request #%s') % c.pull_request.pull_request_id}
4 ${c.repo_name} ${_('Pull request #%s') % c.pull_request.pull_request_id}
5 </%def>
5 </%def>
6
6
7 <%def name="breadcrumbs_links()">
7 <%def name="breadcrumbs_links()">
8 ${h.link_to(_(u'Home'),h.url('/'))}
8 ${h.link_to(_(u'Home'),h.url('/'))}
9 &raquo;
9 &raquo;
10 ${h.link_to(c.repo_name,h.url('changelog_home',repo_name=c.repo_name))}
10 ${h.link_to(c.repo_name,h.url('changelog_home',repo_name=c.repo_name))}
11 &raquo;
11 &raquo;
12 ${_('Pull request #%s') % c.pull_request.pull_request_id}
12 ${_('Pull request #%s') % c.pull_request.pull_request_id}
13 </%def>
13 </%def>
14
14
15 <%def name="main()">
15 <%def name="main()">
16
16
17 <div class="box">
17 <div class="box">
18 <!-- box / title -->
18 <!-- box / title -->
19 <div class="title">
19 <div class="title">
20 ${self.breadcrumbs()}
20 ${self.breadcrumbs()}
21 </div>
21 </div>
22 %if c.pull_request.is_closed():
22 %if c.pull_request.is_closed():
23 <div style="padding:10px; font-size:22px;width:100%;text-align: center; color:#88D882">${_('Closed %s') % (h.age(c.pull_request.updated_on))} ${_('with status %s') % h.changeset_status_lbl(c.current_changeset_status)}</div>
23 <div style="padding:10px; font-size:22px;width:100%;text-align: center; color:#88D882">${_('Closed %s') % (h.age(c.pull_request.updated_on))} ${_('with status %s') % h.changeset_status_lbl(c.current_changeset_status)}</div>
24 %endif
24 %endif
25 <h3>${_('Title')}: ${c.pull_request.title}</h3>
25 <h3>${_('Title')}: ${c.pull_request.title}</h3>
26
26
27 <div class="form">
27 <div class="form">
28 <div id="summary" class="fields">
28 <div id="summary" class="fields">
29 <div class="field">
29 <div class="field">
30 <div class="label-summary">
30 <div class="label-summary">
31 <label>${_('Status')}:</label>
31 <label>${_('Status')}:</label>
32 </div>
32 </div>
33 <div class="input">
33 <div class="input">
34 <div class="changeset-status-container" style="float:none;clear:both">
34 <div class="changeset-status-container" style="float:none;clear:both">
35 %if c.current_changeset_status:
35 %if c.current_changeset_status:
36 <div title="${_('Pull request status')}" class="changeset-status-lbl">[${h.changeset_status_lbl(c.current_changeset_status)}]</div>
36 <div title="${_('Pull request status')}" class="changeset-status-lbl">[${h.changeset_status_lbl(c.current_changeset_status)}]</div>
37 <div class="changeset-status-ico" style="padding:1px 4px"><img src="${h.url('/images/icons/flag_status_%s.png' % c.current_changeset_status)}" /></div>
37 <div class="changeset-status-ico" style="padding:1px 4px"><img src="${h.url('/images/icons/flag_status_%s.png' % c.current_changeset_status)}" /></div>
38 %endif
38 %endif
39 </div>
39 </div>
40 </div>
40 </div>
41 </div>
41 </div>
42 <div class="field">
42 <div class="field">
43 <div class="label-summary">
43 <div class="label-summary">
44 <label>${_('Still not reviewed by')}:</label>
44 <label>${_('Still not reviewed by')}:</label>
45 </div>
45 </div>
46 <div class="input">
46 <div class="input">
47 % if len(c.pull_request_pending_reviewers) > 0:
47 % if len(c.pull_request_pending_reviewers) > 0:
48 <div class="tooltip" title="${h.tooltip(','.join([x.username for x in c.pull_request_pending_reviewers]))}">${ungettext('%d reviewer', '%d reviewers',len(c.pull_request_pending_reviewers)) % len(c.pull_request_pending_reviewers)}</div>
48 <div class="tooltip" title="${h.tooltip(','.join([x.username for x in c.pull_request_pending_reviewers]))}">${ungettext('%d reviewer', '%d reviewers',len(c.pull_request_pending_reviewers)) % len(c.pull_request_pending_reviewers)}</div>
49 %else:
49 %else:
50 <div>${_('pull request was reviewed by all reviewers')}</div>
50 <div>${_('pull request was reviewed by all reviewers')}</div>
51 %endif
51 %endif
52 </div>
52 </div>
53 </div>
53 </div>
54 <div class="field">
54 <div class="field">
55 <div class="label-summary">
55 <div class="label-summary">
56 <label>${_('Origin repository')}:</label>
56 <label>${_('Origin repository')}:</label>
57 </div>
57 </div>
58 <div class="input">
58 <div class="input">
59 <div>
59 <div>
60 ##%if h.is_hg(c.pull_request.org_repo):
60 ##%if h.is_hg(c.pull_request.org_repo):
61 ## <img class="icon" title="${_('Mercurial repository')}" alt="${_('Mercurial repository')}" src="${h.url('/images/icons/hgicon.png')}"/>
61 ## <img class="icon" title="${_('Mercurial repository')}" alt="${_('Mercurial repository')}" src="${h.url('/images/icons/hgicon.png')}"/>
62 ##%elif h.is_git(c.pull_request.org_repo):
62 ##%elif h.is_git(c.pull_request.org_repo):
63 ## <img class="icon" title="${_('Git repository')}" alt="${_('Git repository')}" src="${h.url('/images/icons/giticon.png')}"/>
63 ## <img class="icon" title="${_('Git repository')}" alt="${_('Git repository')}" src="${h.url('/images/icons/giticon.png')}"/>
64 ##%endif
64 ##%endif
65 <span class="spantag">${c.pull_request.org_ref_parts[0]}</span>
65 <span class="spantag">${c.pull_request.org_ref_parts[0]}</span>
66 :
66 :
67 <span class="spantag">${c.pull_request.org_ref_parts[1]}</span>
67 <span class="spantag">${c.pull_request.org_ref_parts[1]}</span>
68 <span><a href="${h.url('summary_home', repo_name=c.pull_request.org_repo.repo_name)}">${c.pull_request.org_repo.clone_url()}</a></span>
68 <span><a href="${h.url('summary_home', repo_name=c.pull_request.org_repo.repo_name)}">${c.pull_request.org_repo.clone_url()}</a></span>
69 </div>
69 </div>
70 </div>
70 </div>
71 </div>
71 </div>
72 <div class="field">
73 <div class="label-summary">
74 <label>${_('Summary')}:</label>
75 </div>
76 <div class="input">
77 <div style="white-space:pre-wrap">${h.literal(c.pull_request.description)}</div>
78 </div>
79 </div>
80 <div class="field">
81 <div class="label-summary">
82 <label>${_('Created on')}:</label>
83 </div>
84 <div class="input">
85 <div>${h.fmt_date(c.pull_request.created_on)}</div>
86 </div>
87 </div>
72 </div>
88 </div>
73 </div>
89 </div>
74 <div style="white-space:pre-wrap;padding:3px 3px 5px 20px">${h.literal(c.pull_request.description)}</div>
75 <div style="padding:4px 4px 10px 20px">
76 <div>${_('Created on')}: ${h.fmt_date(c.pull_request.created_on)}</div>
77 </div>
78
90
79 <div style="overflow: auto;">
91 <div style="overflow: auto;">
80 ##DIFF
92 ##DIFF
81 <div class="table" style="float:left;clear:none">
93 <div class="table" style="float:left;clear:none">
82 <div id="body" class="diffblock">
94 <div id="body" class="diffblock">
83 <div style="white-space:pre-wrap;padding:5px">${_('Compare view')}</div>
95 <div style="white-space:pre-wrap;padding:5px">${_('Compare view')}</div>
84 </div>
96 </div>
85 <div id="changeset_compare_view_content">
97 <div id="changeset_compare_view_content">
86 ##CS
98 ##CS
87 <div style="font-size:1.1em;font-weight: bold;clear:both;padding-top:10px">${ungettext('Showing %s commit','Showing %s commits', len(c.cs_ranges)) % len(c.cs_ranges)}</div>
99 <div style="font-size:1.1em;font-weight: bold;clear:both;padding-top:10px">${ungettext('Showing %s commit','Showing %s commits', len(c.cs_ranges)) % len(c.cs_ranges)}</div>
88 <%include file="/compare/compare_cs.html" />
100 <%include file="/compare/compare_cs.html" />
89
101
90 ## FILES
102 ## FILES
91 <div style="font-size:1.1em;font-weight: bold;clear:both;padding-top:10px">
103 <div style="font-size:1.1em;font-weight: bold;clear:both;padding-top:10px">
92
104
93 % if c.limited_diff:
105 % if c.limited_diff:
94 ${ungettext('%s file changed', '%s files changed', len(c.files)) % len(c.files)}
106 ${ungettext('%s file changed', '%s files changed', len(c.files)) % len(c.files)}
95 % else:
107 % else:
96 ${ungettext('%s file changed with %s insertions and %s deletions','%s files changed with %s insertions and %s deletions', len(c.files)) % (len(c.files),c.lines_added,c.lines_deleted)}:
108 ${ungettext('%s file changed with %s insertions and %s deletions','%s files changed with %s insertions and %s deletions', len(c.files)) % (len(c.files),c.lines_added,c.lines_deleted)}:
97 %endif
109 %endif
98
110
99 </div>
111 </div>
100 <div class="cs_files">
112 <div class="cs_files">
101 %if not c.files:
113 %if not c.files:
102 <span class="empty_data">${_('No files')}</span>
114 <span class="empty_data">${_('No files')}</span>
103 %endif
115 %endif
104 %for fid, change, f, stat in c.files:
116 %for fid, change, f, stat in c.files:
105 <div class="cs_${change}">
117 <div class="cs_${change}">
106 <div class="node">${h.link_to(h.safe_unicode(f),h.url.current(anchor=fid))}</div>
118 <div class="node">${h.link_to(h.safe_unicode(f),h.url.current(anchor=fid))}</div>
107 <div class="changes">${h.fancy_file_stats(stat)}</div>
119 <div class="changes">${h.fancy_file_stats(stat)}</div>
108 </div>
120 </div>
109 %endfor
121 %endfor
110 </div>
122 </div>
111 % if c.limited_diff:
123 % if c.limited_diff:
112 <h5>${_('Changeset was too big and was cut off...')}</h5>
124 <h5>${_('Changeset was too big and was cut off...')}</h5>
113 % endif
125 % endif
114 </div>
126 </div>
115 </div>
127 </div>
116 ## REVIEWERS
128 ## REVIEWERS
117 <div style="float:left; border-left:1px dashed #eee">
129 <div style="float:left; border-left:1px dashed #eee">
118 <h4>${_('Pull request reviewers')}</h4>
130 <h4>${_('Pull request reviewers')}</h4>
119 <div id="reviewers" style="padding:0px 0px 5px 10px">
131 <div id="reviewers" style="padding:0px 0px 5px 10px">
120 ## members goes here !
132 ## members goes here !
121 <div class="group_members_wrap" style="min-height:45px">
133 <div class="group_members_wrap" style="min-height:45px">
122 <ul id="review_members" class="group_members">
134 <ul id="review_members" class="group_members">
123 %for member,status in c.pull_request_reviewers:
135 %for member,status in c.pull_request_reviewers:
124 <li id="reviewer_${member.user_id}">
136 <li id="reviewer_${member.user_id}">
125 <div class="reviewers_member">
137 <div class="reviewers_member">
126 <div style="float:left;padding:0px 3px 0px 0px" class="tooltip" title="${h.tooltip(h.changeset_status_lbl(status[0][1].status if status else 'not_reviewed'))}">
138 <div style="float:left;padding:0px 3px 0px 0px" class="tooltip" title="${h.tooltip(h.changeset_status_lbl(status[0][1].status if status else 'not_reviewed'))}">
127 <img src="${h.url(str('/images/icons/flag_status_%s.png' % (status[0][1].status if status else 'not_reviewed')))}"/>
139 <img src="${h.url(str('/images/icons/flag_status_%s.png' % (status[0][1].status if status else 'not_reviewed')))}"/>
128 </div>
140 </div>
129 <div class="gravatar"><img alt="gravatar" src="${h.gravatar_url(member.email,14)}"/> </div>
141 <div class="gravatar"><img alt="gravatar" src="${h.gravatar_url(member.email,14)}"/> </div>
130 <div style="float:left">${member.full_name} (${_('owner') if c.pull_request.user_id == member.user_id else _('reviewer')})</div>
142 <div style="float:left">${member.full_name} (${_('owner') if c.pull_request.user_id == member.user_id else _('reviewer')})</div>
131 <input type="hidden" value="${member.user_id}" name="review_members" />
143 <input type="hidden" value="${member.user_id}" name="review_members" />
132 %if not c.pull_request.is_closed() and (h.HasPermissionAny('hg.admin', 'repository.admin')() or c.pull_request.user_id == c.rhodecode_user.user_id):
144 %if not c.pull_request.is_closed() and (h.HasPermissionAny('hg.admin', 'repository.admin')() or c.pull_request.user_id == c.rhodecode_user.user_id):
133 <span class="delete_icon action_button" onclick="removeReviewer(${member.user_id})"></span>
145 <span class="delete_icon action_button" onclick="removeReviewer(${member.user_id})"></span>
134 %endif
146 %endif
135 </div>
147 </div>
136 </li>
148 </li>
137 %endfor
149 %endfor
138 </ul>
150 </ul>
139 </div>
151 </div>
140 %if not c.pull_request.is_closed():
152 %if not c.pull_request.is_closed():
141 <div class='ac'>
153 <div class='ac'>
142 %if h.HasPermissionAny('hg.admin', 'repository.admin')() or c.pull_request.author.user_id == c.rhodecode_user.user_id:
154 %if h.HasPermissionAny('hg.admin', 'repository.admin')() or c.pull_request.author.user_id == c.rhodecode_user.user_id:
143 <div class="reviewer_ac">
155 <div class="reviewer_ac">
144 ${h.text('user', class_='yui-ac-input')}
156 ${h.text('user', class_='yui-ac-input')}
145 <span class="help-block">${_('Add reviewer to this pull request.')}</span>
157 <span class="help-block">${_('Add reviewer to this pull request.')}</span>
146 <div id="reviewers_container"></div>
158 <div id="reviewers_container"></div>
147 </div>
159 </div>
148 <div style="padding:0px 10px">
160 <div style="padding:0px 10px">
149 <span id="update_pull_request" class="ui-btn xsmall">${_('save')}</span>
161 <span id="update_pull_request" class="ui-btn xsmall">${_('save')}</span>
150 </div>
162 </div>
151 %endif
163 %endif
152 </div>
164 </div>
153 %endif
165 %endif
154 </div>
166 </div>
155 </div>
167 </div>
156 </div>
168 </div>
157 <script>
169 <script>
158 var _USERS_AC_DATA = ${c.users_array|n};
170 var _USERS_AC_DATA = ${c.users_array|n};
159 var _GROUPS_AC_DATA = ${c.users_groups_array|n};
171 var _GROUPS_AC_DATA = ${c.users_groups_array|n};
160 AJAX_COMMENT_URL = "${url('pullrequest_comment',repo_name=c.repo_name,pull_request_id=c.pull_request.pull_request_id)}";
172 AJAX_COMMENT_URL = "${url('pullrequest_comment',repo_name=c.repo_name,pull_request_id=c.pull_request.pull_request_id)}";
161 AJAX_COMMENT_DELETE_URL = "${url('pullrequest_comment_delete',repo_name=c.repo_name,comment_id='__COMMENT_ID__')}";
173 AJAX_COMMENT_DELETE_URL = "${url('pullrequest_comment_delete',repo_name=c.repo_name,comment_id='__COMMENT_ID__')}";
162 AJAX_UPDATE_PULLREQUEST = "${url('pullrequest_update',repo_name=c.repo_name,pull_request_id=c.pull_request.pull_request_id)}"
174 AJAX_UPDATE_PULLREQUEST = "${url('pullrequest_update',repo_name=c.repo_name,pull_request_id=c.pull_request.pull_request_id)}"
163 </script>
175 </script>
164
176
165 ## diff block
177 ## diff block
166 <%namespace name="diff_block" file="/changeset/diff_block.html"/>
178 <%namespace name="diff_block" file="/changeset/diff_block.html"/>
167 %for fid, change, f, stat in c.files:
179 %for fid, change, f, stat in c.files:
168 ${diff_block.diff_block_simple([c.changes[fid]])}
180 ${diff_block.diff_block_simple([c.changes[fid]])}
169 %endfor
181 %endfor
170 % if c.limited_diff:
182 % if c.limited_diff:
171 <h4>${_('Changeset was too big and was cut off...')}</h4>
183 <h4>${_('Changeset was too big and was cut off...')}</h4>
172 % endif
184 % endif
173
185
174
186
175 ## template for inline comment form
187 ## template for inline comment form
176 <%namespace name="comment" file="/changeset/changeset_file_comment.html"/>
188 <%namespace name="comment" file="/changeset/changeset_file_comment.html"/>
177 ${comment.comment_inline_form()}
189 ${comment.comment_inline_form()}
178
190
179 ## render comments and inlines
191 ## render comments and inlines
180 ${comment.generate_comments(include_pr=True)}
192 ${comment.generate_comments(include_pr=True)}
181
193
182 % if not c.pull_request.is_closed():
194 % if not c.pull_request.is_closed():
183 ## main comment form and it status
195 ## main comment form and it status
184 ${comment.comments(h.url('pullrequest_comment', repo_name=c.repo_name,
196 ${comment.comments(h.url('pullrequest_comment', repo_name=c.repo_name,
185 pull_request_id=c.pull_request.pull_request_id),
197 pull_request_id=c.pull_request.pull_request_id),
186 c.current_changeset_status,
198 c.current_changeset_status,
187 close_btn=True, change_status=c.allowed_to_change_status)}
199 close_btn=True, change_status=c.allowed_to_change_status)}
188 %endif
200 %endif
189
201
190 <script type="text/javascript">
202 <script type="text/javascript">
191 YUE.onDOMReady(function(){
203 YUE.onDOMReady(function(){
192 PullRequestAutoComplete('user', 'reviewers_container', _USERS_AC_DATA, _GROUPS_AC_DATA);
204 PullRequestAutoComplete('user', 'reviewers_container', _USERS_AC_DATA, _GROUPS_AC_DATA);
193
205
194 YUE.on(YUQ('.show-inline-comments'),'change',function(e){
206 YUE.on(YUQ('.show-inline-comments'),'change',function(e){
195 var show = 'none';
207 var show = 'none';
196 var target = e.currentTarget;
208 var target = e.currentTarget;
197 if(target.checked){
209 if(target.checked){
198 var show = ''
210 var show = ''
199 }
211 }
200 var boxid = YUD.getAttribute(target,'id_for');
212 var boxid = YUD.getAttribute(target,'id_for');
201 var comments = YUQ('#{0} .inline-comments'.format(boxid));
213 var comments = YUQ('#{0} .inline-comments'.format(boxid));
202 for(c in comments){
214 for(c in comments){
203 YUD.setStyle(comments[c],'display',show);
215 YUD.setStyle(comments[c],'display',show);
204 }
216 }
205 var btns = YUQ('#{0} .inline-comments-button'.format(boxid));
217 var btns = YUQ('#{0} .inline-comments-button'.format(boxid));
206 for(c in btns){
218 for(c in btns){
207 YUD.setStyle(btns[c],'display',show);
219 YUD.setStyle(btns[c],'display',show);
208 }
220 }
209 })
221 })
210
222
211 YUE.on(YUQ('.line'),'click',function(e){
223 YUE.on(YUQ('.line'),'click',function(e){
212 var tr = e.currentTarget;
224 var tr = e.currentTarget;
213 injectInlineForm(tr);
225 injectInlineForm(tr);
214 });
226 });
215
227
216 // inject comments into they proper positions
228 // inject comments into they proper positions
217 var file_comments = YUQ('.inline-comment-placeholder');
229 var file_comments = YUQ('.inline-comment-placeholder');
218 renderInlineComments(file_comments);
230 renderInlineComments(file_comments);
219
231
220 YUE.on(YUD.get('update_pull_request'),'click',function(e){
232 YUE.on(YUD.get('update_pull_request'),'click',function(e){
221 updateReviewers();
233 updateReviewers();
222 })
234 })
223 })
235 })
224 </script>
236 </script>
225
237
226 </div>
238 </div>
227
239
228 </%def>
240 </%def>
General Comments 0
You need to be logged in to leave comments. Login now