CatEventHandler fatal error when ItemPropertyMappings not defined

[Sergejs Gribs]

OnBeforeItemUpdate event in CatEventHandler class throws fatal error when trying to access undefined variable as object. This happen because $object is defined inside "if" construction, but after that is also used outside. Nobody never tried making custom Category Item without ItemPropertyMappings defined in config (this is "if" condition) until now, so we never have seen this issue.

Attached patch fixes the bug.

[Alexander Obuhovich]

Such type of error is really hard to pinpoint. The only way is by using problematic code, which you actually did. Thanks for the input.

On Wed, Oct 28, 2009 at 10:39 PM, Sergejs Gribs <serg...@gmail.com> wrote:

OnBeforeItemUpdate event in CatEventHandler class throws fatal error when trying to access undefined variable as object. This happen because $object is defined inside "if" construction, but after that is also used outside. Nobody never tried making custom Category Item without ItemPropertyMappings defined in config (this is "if" condition) until now, so we never have seen this issue.

Attached patch fixes the bug.

--
Best Regards,

http://www.alex-time.com

[Dmitry A.]

New bug filed.

440: CatEventHandler fatal error when ItemPropertyMappings not defined

http://tracker.in-portal.org/view.php?id=440


DA.

On Oct 29, 1:06 pm, Alexander Obuhovich <aik.b...@gmail.com> wrote:
> Such type of error is really hard to pinpoint. The only way is by using
> problematic code, which you actually did. Thanks for the input.
>