Hi Greg, Thanks for detailed explanation. I will make destroyErrorTip protected.
Piotr śr., 8 maj 2019 o 22:46 Greg Dove <[email protected]> napisał(a): > Hi Piotr, > > I remember a bit more now. I believe this was related to error tips not > being removed when a view state that 'owns' them changes or there is some > need to remove them like that other than via the Validator's internal code. > I added a utility method into FormValidator which does the hard work of > iterating validators it finds and removing errorTips, but iirc this was > intended to be available for individual Validators. The alternative was > making the protected method public which did not seem right to me at the > time. By making it internal, I could 'export' it via a utility function so > that it was available externally in the rare cases that needed it. > I agree it is unconventional, and probably it was never the right solution. > It looks like Carlos added a new approach near middile of March which is > event driven using 'cleanValidationErrors' to support that type of thing, > so if you are happy to make that change back to protected, I'd suggest go > ahead, or let me know if you want to do it (I will probably be 1 day or 2 > before I can do that). I will need to make parallel changes in Carlos' > project, which has a number of custom validators, and which uses the > exported utility function for override like behaviour. But that will be > easy to switch back - just a simple replace of removeErrorTip(this) to > destroyErrorTip() once it is back to protected instead of internal, I > think. > > > > > > On Thu, May 9, 2019 at 3:39 AM Greg Dove <[email protected]> wrote: > > > That was me. Iirc it was to make it possible to remove error tips from > > other code elsewhere... see the importable utility function in the > package > > and search for where it is being used. I used this inside some subclasses > > too to achieve 'override' while still making that support available > > elsewhere. Maybe you can find a better way.... sorry just a quick > reply... > > 3.30 am for me. I should be asleep! Will look more tomorrow and give more > > detail if needed. > > > > On Thu, 9 May 2019, 03:24 Carlos Rovira, <[email protected]> > > wrote: > > > >> Hi Piotr, > >> > >> I'm surprised too. Maybe this change was done by Harbs blur listener / > >> removeTip > >> > >> Hope he can let us know the reason > >> > >> > >> > >> El mié., 8 may. 2019 a las 14:48, Piotr Zarzycki (< > >> [email protected]>) > >> escribió: > >> > >> > Hi Carlos, > >> > > >> > I have found that I cannot use destroyErrorTip in my project cause > it's > >> > internal. Is there any reason why it is not protected ? > >> > > >> > Thanks, > >> > -- > >> > > >> > Piotr Zarzycki > >> > > >> > Patreon: *https://www.patreon.com/piotrzarzycki > >> > <https://www.patreon.com/piotrzarzycki>* > >> > > >> > >> > >> -- > >> > >> <http://www.codeoscopic.com> > >> > >> Carlos Rovira > >> > >> Presidente Ejecutivo > >> > >> M: +34 607 22 60 05 > >> > >> http://www.codeoscopic.com > >> > >> > >> Conócenos en 1 minuto! <https://avant2.es/#video> > >> > >> > >> AVISO LEGAL: La información contenida en este correo electrónico, y en > su > >> caso en los documentos adjuntos, es información privilegiada para uso > >> exclusivo de la persona y/o personas a las que va dirigido. No está > >> permitido el acceso a este mensaje a cualquier otra persona distinta a > los > >> indicados. Si Usted no es uno de los destinatarios, cualquier > duplicación, > >> reproducción, distribución, así como cualquier uso de la información > >> contenida en él o cualquiera otra acción u omisión tomada en relación > con > >> el mismo, está prohibida y puede ser ilegal. En dicho caso, por favor, > >> notifíquelo al remitente y proceda a la eliminación de este correo > >> electrónico, así como de sus adjuntos si los hubiere. En cumplimiento de > >> la > >> legislación española vigente en materia de protección de datos de > carácter > >> personal y del RGPD 679/2016 le informamos que sus datos están siendo > >> objeto de tratamiento por parte de CODEOSCOPIC S.A. con CIFA85677342, > con > >> la finalidad del mantenimiento y gestión de relaciones comerciales y > >> administrativas. La base jurídica del tratamiento es el interés legítimo > >> de > >> la empresa. No se prevén cesiones de sus datos, salvo que exista una > >> obligación legal. Para ejercitar sus derechos puede dirigirse a > >> CODEOSCOPIC > >> S.A., domiciliada enPaseo de la Habana, 9-11, 28036 de Madrid (MADRID), > o > >> bien por email [email protected], con el fin de ejercer sus derechos > >> de > >> acceso, rectificación, supresión (derecho al olvido), limitación de > >> tratamiento, portabilidad de los datos, oposición, y a no ser objeto de > >> decisiones automatizadas, indicando como Asunto: “Derechos Ley > Protección > >> de Datos”, y adjuntando fotocopia de su DNI. Delegado de protección de > >> datos:[email protected] > >> > > > -- Piotr Zarzycki Patreon: *https://www.patreon.com/piotrzarzycki <https://www.patreon.com/piotrzarzycki>*
