beberlei / zf-doctrine

A Zend Framework 1.x and Doctrine 1.2 Integration - UNMAINTAINED
101 stars 21 forks source link

Validator ZFDoctrine_Validate_Unique #24

Open data219 opened 13 years ago

data219 commented 13 years ago

Hallo Benny,

ich habe einen schicken neuen Unique-Constraint-Validator gebastelt. ModelForm mußte angepaßt werden, so dass der Validator gefunden werden kann. Der Action-Helper muß nun alle Parameter an das Form übergeben (nicht drastisch, da das Form ja eh nur die verarbeitet zu denen es Felder hat), so dass der Kontext der an die Validatoren übergeben wird auch die GET-Parameter beinhaltet und damit auch ggf den PrimaryKey, den benötigt der Validator damit er checken kann ob nicht gerade ein Update läuft und der gefundene Record daher die Unique-Constraint garnicht verletzt. (Wenn nicht klar was ich meine versuch einfach mal den Validator ohne die Anpassung am Action-Helper und ändere einen bestehenden Record ;) ) Da ich den Update-Fall zuerst total vergessen hatte sind es leider direkt 2 Commits. (Und ein vorheriger Pull-Request den ich wieder zurück gezogen habe). Wäre schön wenn es Deinen Zuspruch findet, dann kann ich in der Installtionsanweisung auf dein Repo verweisen und muss dort nicht meinen Fork eintragen und den auf ewig am Leben halten. Wenn es Dir eher nicht zusagt geb mir doch bitte Bescheid, dann werd ich versuchen die Änderungen in Ableitungen zu packen und das ganze in einem anderen Namensraum in meinem Projekt zu verlagern.

LG Jan

daffl commented 13 years ago

Vielleicht bin ich ein bischen zu spät, aber ich möchte mit meinem Fork alle Doctrine Validatoren unterstützen:

https://github.com/daffl/zf-doctrine

Können wir das irgendwie zusammenbringen?

data219 commented 13 years ago

Klar können wir das, da du mehr Validatoren hast und ich nur einen magst einfach meinen bei dir reinmergen? wobei vielleicht warten wir mal ob Benni Zeit findet sich den Kram anzuschauen...