Description: ------------ When you use crosslinks with FormBuilder and make updates to the crosslinked entries, FB will delete all previous entries in the crosslink table and re-insert the checked entries anew. This behaviour is okay as long as you store only the linked fields in the crosslink table. When you put in additional information, however, this information is being lost whenever you add or remove an option from the crosslink table, which is not desirable. Sönke Ruempler has provided a patch to fix this, which can be found here: http://opensource.21st.de/projects/formbuilder/patches/FormBuilder_SRuempler.phps

I've made a patch vs. the current CVS version. Seems to work just fine. It's more overhead, but it should work ok. Patch here: http://www.reversefold.com/PEAR/FB_bug2841.patch

Ugh...the new bug link thing killed my link! http://www.reversefold.com/PEAR/FB_b2841.patch

This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better. I've made some changes to the original patch. It's a bit more efficient and also syncs the triplelinks.

Anyone following this bug may want to check out bug #2966.

Justin, there is a bug that I did not notice in the patch: if no checkbox is checked, there is not var/array from the outside and foreach throws warnings, should be fixed with som,ething like this: $fieldvalues = &$values['__crossLink_'.$crossLink['table']]; if (!is_array($fieldvalues)) { $fieldvalues = array(); } soenke

Thanks for the update. That solution, however, will cause a notice if the $values entry isn't set. This should be fixed in CVS.

Thx for the fix Justin, but there should be no E_NOTICE, because I assigned the value per reference ;-) Happy X-Mas ;-)

I had no idea that assigning by ref got around the "variable not set" or "index not set" notices. Still, I'd see that as a hack. This feels cleaner (and makes more sense) to me: if (isset($values['__crossLink_'.$crossLink['table']])) { $fieldvalues = $values['__crossLink_'.$crossLink['table']]; } else { $fieldvalues = array(); }