* code for task#1844.

module/tree/model.php

@@ -459,12 +459,11 @@ class treeModel extends model
                     }
                     $menu .= '</ul></li>';
                 }
-
                 $menu .= '</ul>';
             }
+            $menu .='</li>';
         }
-
-        $menu .= '</li>';
+        $menu .= '</ul>';
         return $menu;
     }
 

module/tree/view/browse.html.php

@@ -56,7 +56,7 @@
                   if($allProject)
                   {
                       echo "<span class='copy'>";
-                      echo html::select('allProject', $allProject, '', 'onchange=syncProductOrProject(this,"project")');
+                      echo html::select('allProject', $allProject, '', "onchange=\"syncProductOrProject(this,'project')\"");
                       echo html::select('projectModule', $projectModules, '');
                       echo html::commonButton($lang->tree->syncFromProject, "id='copyModule' onclick='syncModule($currentProject, \"task\")'");
                       echo '</span>';
@@ -67,7 +67,7 @@
                   if($allProduct)
                   {
                       echo "<span class='copy'>";
-                      echo html::select('allProduct', $allProduct, '', 'onchange=syncProductOrProject(this,"product")');
+                      echo html::select('allProduct', $allProduct, '', "onchange=\"syncProductOrProject(this,'product')\"");
                       echo html::select('productModule', $productModules, '');
                       echo html::commonButton($lang->tree->syncFromProduct, "id='copyModule' onclick='syncModule($currentProduct, \"story\")'");
                       echo '</span>';