Something really odd: function sort_table_first($a, $b) is defined inside public function action_install(). There are more like that in Packages.php
Technically it's allowed but do we want this? IMO it isn't really good practice..
I tend to agree... They were probably not meant to be reused so defined where needed, but we don't need to continue the practice methinks.