Para reproducir el error, se debe tener un socio con una restricción, ya sea manual o automática: 1- Editar el socio 2- Marcar el checkbox "eliminar" en la restricción 3- Guardar En ese momento aparece un internal server error En los logs el mensaje es: ==> /var/log/koha/eco/plack-error.log <== No property remove_debarment for Koha::Patron La restricción se elimina, pero se debe abrir una ventana de koha nuevamente.
Hola Frederico, no puedo reproducir este problema. Cual versión usas? Reiniciaste Plack? I cannot recreate this issue, which version are you using? Did you restart Plack?
Reinicie plack y sigue saliendo el mismo error. Utilizo debian 8 y koha está en su viersión 18.11.05.000
I've added adding and deleting restrictions from a patron account in staff in 18.11 and it worked ok, no internal server error. Not sure I tested the right thing, can someone help?
Federico, can you locate where you Koha code is (should be something like /usr/share/koha/lib) then copy the output of the following command: % grep -r remove_debarment
there is no output for the command
(In reply to Federico Rinaudo from comment #5) > there is no output for the command It might come from the templates. Try: $ cd /usr/share/koha $ grep -r remove_debarment
/usr/share/koha# grep -r remove_debarment intranet/cgi-bin/members/memberentry.pl:my @debarments_to_remove = $input->multi_param('remove_debarment'); intranet/htdocs/intranet-tmpl/prog/en/modules/members/memberentrygen.tt: <input type="checkbox" id="debarment_[% d.borrower_debarment_id | html %]" name="remove_debarment" value="[% d.borrower_debarment_id | html %]" /> intranet/htdocs/intranet-tmpl/prog/es-ES/modules/members/memberentrygen.tt: <input type="checkbox" id="debarment_[% d.borrower_debarment_id | html %]" name="remove_debarment" value="[% d.borrower_debarment_id | html %]" />
This is correct Federico, I have no idea what is going wrong. If you have a test server you could try to apply this debug statement: @ Object.pm:240 @ sub set { foreach my $p ( keys %$properties ) { unless ( grep {/^$p$/} @columns ) { + my @c = caller; use Data::Printer colored => 1; warn p @c; Koha::Exceptions::Object::PropertyNotFound->throw( "No property $p for " . ref($self) ); } } To understand where it comes from. Or, you can turn the dev_install flag on in your koha-conf.xml file, then restart plack. You will get a full stack trace of the error which will be easier to track the problem down.
I'm able to recreate this on 18.11.06. Plack log showing the same error "No property remove_debarment for Koha::Patron." When I reload the patron the debarment has been removed, despite the error
Oh, and I do not get this error when removing the debarment from the restrictions tab on circulation.pl. Only when removing the debarment from memberentry.pl
(In reply to Andrew from comment #10) > Oh, and I do not get this error when removing the debarment from the > restrictions tab on circulation.pl. Only when removing the debarment from > memberentry.pl That was the useful bit, thanks! Confirmed.
Created attachment 90954 [details] [review] Bug 23082: FIX debarment removing when editing a patron In members/memberentry.pl we have to explicitely remove the template's params that are not patron's attributes. Certainly caused by commit 1bb6cec902088d07c7a43addc4b4adb5b0bf3d4f Bug 20287: Fix update of patrons, clean the data before ->store Test plan: Create a restriction for a patron Edit patron's details Remove the restriction => Without this patch you get "No property remove_debarment for Koha::Patron" => With this patch applied the restriction is removed
Created attachment 90958 [details] [review] Bug 23082: FIX debarment removing when editing a patron In members/memberentry.pl we have to explicitely remove the template's params that are not patron's attributes. Certainly caused by commit 1bb6cec902088d07c7a43addc4b4adb5b0bf3d4f Bug 20287: Fix update of patrons, clean the data before ->store Test plan: Create a restriction for a patron Edit patron's details Remove the restriction => Without this patch you get "No property remove_debarment for Koha::Patron" => With this patch applied the restriction is removed Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Created attachment 90963 [details] [review] Bug 23082: Fatal error editing a restricted patron In members/memberentry.pl we have to explicitely remove the template's params that are not patron's attributes. Certainly caused by commit 1bb6cec902088d07c7a43addc4b4adb5b0bf3d4f Bug 20287: Fix update of patrons, clean the data before ->store Test plan: Create a restriction for a patron Edit patron's details Remove the restriction => Without this patch you get "No property remove_debarment for Koha::Patron" => With this patch applied the restriction is removed Signed-off-by: Mark Tompsett <mtompset@hotmail.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Nice work! Pushed to master for 19.11.00
Pushed to 19.05.x for 19.05.01
backported to 18.11.x for 18.11.07
*** Bug 23972 has been marked as a duplicate of this bug. ***