There are multiple validation needs inside Koha and currently there is a lot of code duplication regarding for ex simple data type validation (double, integer, char, string, array of emails, ...) Create a basis for Koha-wide validations usable dynamically with exceptions or without.
Created attachment 56266 [details] [review] Bug 17433 - Koha::Validation. Validation for the masses! Koha::Validation->tries now supports data structure validation and some basic type validations.
What do you expect the community to do with this patch exactly?
(In reply to Jonathan Druart from comment #2) > What do you expect the community to do with this patch exactly? Do you have another kind of validation system in Koha? Do you think validation of variables is not important?
You do not answer the question :) The status is new it and depends on a blocked bug report (which depends on another blocked).
any updates on bug 17433? Carnegie team
(In reply to Brian from comment #5) > any updates on bug 17433? > > Carnegie team Why are you asking? What are you expecting from this patch?
I just know that it is related to at least 3 other patches of a ticket that concerns a problem with the Validation for *MandatoryField and *UnwantedField sysprefs. For now I'm putting my foot in the door as to speak so I receive email updates when progress is made and hopefully the patch is eventually finished
(In reply to Brian from comment #7) > I just know that it is related to at least 3 other patches of a ticket that > concerns a problem with the Validation for *MandatoryField and > *UnwantedField sysprefs. For now I'm putting my foot in the door as to > speak so I receive email updates when progress is made and hopefully the > patch is eventually finished There are no other bugs linked to this, so you probably will not receive any emails. This is at the moment a bit of a dead end.
(In reply to Katrin Fischer from comment #8) > (In reply to Brian from comment #7) > > I just know that it is related to at least 3 other patches of a ticket that > > concerns a problem with the Validation for *MandatoryField and > > *UnwantedField sysprefs. For now I'm putting my foot in the door as to > > speak so I receive email updates when progress is made and hopefully the > > patch is eventually finished > > There are no other bugs linked to this, so you probably will not receive any > emails. This is at the moment a bit of a dead end. Filing a new separate omnibus bug for validation issues and link them there could be a good idea. I just wouldn't use this here as development seems to have stopped.
A similar idea on bug 34430