https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42128
--- Comment #7 from Jonathan Druart <[email protected]> --- Not coming from this patch, but the following code is not nice: 75 [% IF image_location.match('^http') %] 76 <input type="radio" id="remote_image_check" name="image" value="remoteImage" checked="checked" /> 77 <input id="remoteimage" type="text" name="remoteImage" size="48" maxlength="200" value="[% image_location | html %]" /> 78 <img src="[% image_location | html %]" alt="" /> 79 [% ELSE %] 80 <input type="radio" id="remote_image_check" name="image" value="remoteImage" /> 81 <input type="text" id="remoteimage" name="remoteImage" size="48" maxlength="200" value="" /> 82 [% END %] 83 [% END %] We hide the value if what is in the DB does not start with "http". If you save, you lose the value. We should instead prevent to save incorrect value, not dropping the data without even a warning. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
