Tickets :: Comment on [#11124] decode_attribute hook an exceptions

* Your Email Address
* Spam protection	Enter the letters below: . .. ..__.. ,.__ \| \|\| \|\| \| \./ [__) \|/\\|\|/\\|\|__\| \| [__)
Comment	>>> To get the pictures in 'localsql' back, i have to throw an exception >>> instead of the >>> default passthrough 'return $value'. >> >> Yes, because that's how we detect if an attribute was not processed >> by a hook, and thus continue the further processing of composite and >> image fields further down in getValue(). If you moved the hook >> calling further down, you wouldn't be able to hook into those fields >> at all. >> >>> Now it works as expected, but i have tons of "No decode handler" >>> errors in the Logs. >>> What I'm doing wrong here? >> >> Nothing, though I admit that having the logs spammed with false >> warnings isn't optimal. > > Just a few thoughts: > > - to detect, whether or not an attribute has processed, one could > compare the value before and after the hook > - is it really desired, to have composite fields processed by the hook? > They are a combination of driver values (that should be processed) > - are images in this sense not some special kind of composite fields? > >
Attachment
Watch this ticket