Opened 10 years ago
Closed 7 years ago
#31178 closed defect (bug) (duplicate)
make sure that the menu index don't get overridden
Reported by: | pbearne | Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | |
Component: | Plugins | Keywords: | has-patch |
Focuses: | administration | Cc: |
Description
This might not fly but here goes
There is a problem when adding items to the admin menu in that is possible to pick the same index as someone else.
There is protection for the default admin location but this relies on a static array of index.
So I have created patch that solves this by turning the tagart index (60) into 60.12345 $tagart_index. '.' .rand(1 ,1000) and recheck its not used.
I have not changed all the locations needed in the menu and we could apply this to the submenus as well.
For now I am added a function to the menu.php but this might better somewhere else (and renamed)
I wanted to see if it would fly first for doing more work etc.
Paul
Attachments (1)
Note: See
TracTickets for help on using
tickets.
concept patch