kallithea Changeset - 21f80c6cdf0c

Changeset - 21f80c6cdf0c

Parent rev.

Child rev.

[Not reviewed]

default

0 1 0

Takumi IINO - 10 years ago 2015-10-25 19:32:18
trot.thunder@gmail.com

js: wrap repo_switcher setup code in $(document).ready

Without wrapping in a function, some variables were made global, such as cache,
format, visual_show_public_icon.

1 file changed with 2 insertions and 0 deletions:

kallithea/templates/base/base.html

0 comments (0 inline, 0 general)

kallithea/templates/base/base.html

➞

Show inline comments

@@ @@ -344,24 +344,25 @@ @@
               %if not c.authuser.is_external_auth:
                 ## Cannot log out if using external (container) authentication.
                 <li class="logout">${h.link_to(_('Log Out'), h.url('logout_home'))}</li>
               %endif
             </ol>
             </div>
           %endif
         </div>
       </div>
     </li>
     <script type="text/javascript">
         $(document).ready(function(){
         var visual_show_public_icon = "${c.visual.show_public_icon}" == "True";
         var cache = {}
         /*format the look of items in the list*/
         var format = function(state){
             if (!state.id){
               return state.text; // optgroup
+            }
             var obj_dict = state.obj;
             var tmpl = '';
             if(obj_dict && state.type == 'repo'){
                 tmpl += '<span class="repo-icons">';
@@ @@ -434,24 +435,25 @@ @@
                       cache[key] = data;
                       query.callback({results: data.results});
+                    }
                   });
+              }
+            }
         });
         $("#repo_switcher").on('select2-selecting', function(e){
             e.preventDefault();
             window.location = pyroutes.url('summary_home', {'repo_name': e.val});
         });
         });
         ## Global mouse bindings ##
         // general help "?"
         Mousetrap.bind(['?'], function(e) {
             $('#help_kb').modal({});
         });
         // / open the quick filter
         Mousetrap.bind(['/'], function(e) {
             $("#repo_switcher").select2("open");

0 comments (0 inline, 0 general)