Mercurial > kallithea
comparison rhodecode/templates/pullrequests/pullrequest.html @ 3513:a2a61978752f beta
pullrequest: don't do in the ajax callback what can be done in no time right now
author | Mads Kiilerich <madski@unity3d.com> |
---|---|
date | Thu, 07 Mar 2013 19:06:21 +0100 |
parents | ff3f10a6a203 |
children | ba2cbb642c3e |
comparison
equal
deleted
inserted
replaced
3512:ff3f10a6a203 | 3513:a2a61978752f |
---|---|
154 for (k in rev_data){ | 154 for (k in rev_data){ |
155 url = url.replace('__'+k+'__',rev_data[k]); | 155 url = url.replace('__'+k+'__',rev_data[k]); |
156 } | 156 } |
157 | 157 |
158 YUD.get('pull_request_overview').innerHTML = "${_('Loading ...')}"; | 158 YUD.get('pull_request_overview').innerHTML = "${_('Loading ...')}"; |
159 ypjax(url,'pull_request_overview'); | |
160 | |
159 YUD.get('pull_request_overview_url').href = url; // shouldn't have as_form ... but ... | 161 YUD.get('pull_request_overview_url').href = url; // shouldn't have as_form ... but ... |
160 YUD.setStyle(YUD.get('pull_request_overview_url').parentElement,'display',''); | 162 YUD.setStyle(YUD.get('pull_request_overview_url').parentElement,'display',''); |
161 ypjax(url,'pull_request_overview', function(data){ | 163 |
162 var sel_box = YUQ('#pull_request_form #other_repo')[0]; | 164 var sel_box = YUQ('#pull_request_form #other_repo')[0]; |
163 var repo_name = sel_box.options[sel_box.selectedIndex].value; | 165 var repo_name = sel_box.options[sel_box.selectedIndex].value; |
164 YUD.get('other_repo_desc').innerHTML = other_repos_info[repo_name]['description']; | 166 |
165 // replace options of other_ref with the ones for the current other_repo | 167 YUD.get('other_repo_desc').innerHTML = other_repos_info[repo_name]['description']; |
166 var other_ref_selector = YUD.get('other_ref'); | 168 // replace options of other_ref with the ones for the current other_repo |
167 var new_select = YUD.createElementFromMarkup(other_repos_info[repo_name]['revs']); | 169 var other_ref_selector = YUD.get('other_ref'); |
168 var new_selectedIndex = new_select.selectedIndex; | 170 var new_select = YUD.createElementFromMarkup(other_repos_info[repo_name]['revs']); |
169 other_ref_selector.innerHTML = ""; // clear old options | 171 var new_selectedIndex = new_select.selectedIndex; |
170 while (new_select.length > 0){ // children will be popped when appened to other_ref_selector | 172 other_ref_selector.innerHTML = ""; // clear old options |
171 other_ref_selector.appendChild(new_select.children[0]); | 173 while (new_select.length > 0){ // children will be popped when appened to other_ref_selector |
172 } | 174 other_ref_selector.appendChild(new_select.children[0]); |
173 // browsers lost track of selected when appendChild was used | 175 } |
174 other_ref_selector.selectedIndex = new_selectedIndex; | 176 // browsers lost track of selected when appendChild was used |
175 // reset && add the reviewer based on selected repo | 177 other_ref_selector.selectedIndex = new_selectedIndex; |
176 var _data = other_repos_info[repo_name]; | 178 |
177 YUD.get('review_members').innerHTML = ''; | 179 // reset && add the reviewer based on selected repo |
178 addReviewMember(_data.user.user_id, _data.user.firstname, | 180 var _data = other_repos_info[repo_name]; |
179 _data.user.lastname, _data.user.username, | 181 YUD.get('review_members').innerHTML = ''; |
180 _data.user.gravatar_link); | 182 addReviewMember(_data.user.user_id, _data.user.firstname, |
181 }) | 183 _data.user.lastname, _data.user.username, |
184 _data.user.gravatar_link); | |
182 } | 185 } |
183 | 186 |
184 ## refresh automatically when something changes (org_repo can't change) | 187 ## refresh automatically when something changes (org_repo can't change) |
185 | 188 |
186 YUE.on('org_ref', 'change', function(e){ | 189 YUE.on('org_ref', 'change', function(e){ |