{"shas":[["8a1effbe57041d359c9b0bfbc444ce21c41f7321","4b7ab0084a9831295e431077723a463bcb7e7c13@gmail.com","Improve documentation for sort_menu.\n\nCreate docblock. Improve variable names to make the code more self-documenting.","rmburkhead"],["4aae47f3bd102dbb352a4f2daf72ceb0b8b01ee9","4b7ab0084a9831295e431077723a463bcb7e7c13@gmail.com","Force the node IDs processed by max() to be integers. Fixes #3075\n\ngetNodeId() (defined in Zikula.Tree.js) returns a string, therefore the max() operation in genNextId() was returning the max based on string sorting, not on integer sorting. Since \"9\" is greater than \"10\" when sorting string values, only 10 menu entries were ever allowed. Attempting to add an 11th menu entry overwrote the 10th item, even though it appeared on screen that it was properly added. Once saved, the former 10th item disappeared. Adding an 11th, 12th, 13th, etc, etc, and then saving would make the 10th through all but the last item added disappear. Importing a menu initially successfully created a complex menu with more than 10 items, but any attempt to edit and then save would appear to result in random items disappearing from the menu, and possibly also submenus being orphaned and therefore unable to be edited.","rmburkhead"]],"ref":"refs/heads/master","size":2,"head":"4aae47f3bd102dbb352a4f2daf72ceb0b8b01ee9","push_id":33767533}