[Koha-bugs] [Bug 12446] Enable an adult to have a guarantor

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12446

--- Comment #14 from Genevieve Plantin <genevieve.plantin at inlibro.com> ---
Created attachment 42470
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=42470&action=edit
Bug 12446 - Clean code according to Comment 12

For the first recommendation, I tried to be more explicit, I don't know if you
would prefer it because it is slower.

For the second point, I've got rid of the UPDATE line so we don't activate a
new functionnality.

For the third point, I was told that the coma was there in case of new
addition.

Fourth, I corrected the mistake.

For the fifth recommendation, I tried to separate the hardcode categories, but
it is simple for the display (it only changes a the choice in the select for
yes, but the user can still change it aftewards). However, I modified it so, if
someone wants to add a new category to be displayed as guarantee by default, he
only has to add the category code in the array instead of writting another
select == 'new category'.

For the sixth point, I replaced the space.

Lastly, I deleted these lines because, as you said it, it wasn't used.

-- 
You are receiving this mail because:
You are watching all bug changes.