search

jonathangeiger / kohana-jelly

See the link below for the most up-to-date code
https://github.com/creatoro/jelly
MIT License
146 stars 34 forks source link

Double $field->set($value) in Jelly_Model_Core::set() #64

Closed homm closed 14 years ago

homm commented 14 years ago

Is it correct, what $field->set($value) call twice in Jelly_Model_Core::set()?

http://github.com/jonathangeiger/kohana-jelly/blob/master/classes/jelly/model/core.php#L300 http://github.com/jonathangeiger/kohana-jelly/blob/master/classes/jelly/model/core.php#L312

banks commented 14 years ago

No, that looks like a bug. Thanks. Jon's recent branch work may have changed this though so I'll let him resolve it as is best.

jonathangeiger commented 14 years ago

Closed by 5b3ee9e5e553926a28b9b240e3d62ad9c4beea37