The category package has come a long way in removing insert-type functions that duplicate the functionality of update-type functions. The category_menu_entity_inserted() function is one of the last ones that's still hanging around, and it's one of the tougher ones to fix up. But there have been heaps of bugs caused by inconsistencies between this and the category_menu_entity_updated() function. The two functions contain largely duplicate code, with a few special cases that could be handled reasonably well within a single function anyway.

It's time to merge category_menu_entity_inserted() into category_menu_entity_updated(). I'm sick of maintaining both these functions. One of these beasts is more than enough to make any module maintainer go insane. ;-p

Comments

JirkaRybka’s picture

JirkaRybka commented
Status: Active » Fixed

I assume this is just a stale old issue, forgotten in the queue. In 6.x, the function is gone already, after category_menu got rewritten for the new 6.x menu system and for Category 2.0 package. In 4.7.x (version on this issue) and 5.x this change is not going to happen, because these versions are no longer maintained, and are stable releases. So, although I didn't do anything with this myself, I believe I may close this issue safely.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.