Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Validator ZFDoctrine_Validate_Unique #24

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Validator ZFDoctrine_Validate_Unique #24

wants to merge 5 commits into from

Conversation

data219
Copy link

@data219 data219 commented Dec 6, 2010

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
Copy link

daffl commented Dec 23, 2010

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

https:/daffl/zf-doctrine

Können wir das irgendwie zusammenbringen?

@data219
Copy link
Author

data219 commented Jan 4, 2011

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...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants