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

Search Results: 4 of 27 [ <<First <Prev Next> Last>> ] [ Return to Search Results ]

* 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 > > Not easily possible, because the values aren't necessarily scalar values. > >> - is it really desired, to have composite fields processed by the hook? >> They are a combination of driver values (that should be processed) > > This is debatable, but I don't see a reason to artificially limit > which fields we allow to pass through the hook, just for code-flow > reasons. > >> - are images in this sense not some special kind of composite fields? > > Not really.
Attachment
Watch this ticket