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.

Sign up for GitHub

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

Jump to bottom

do not rewrite existing slug #2

Open

anktx wants to merge 2 commits into yiiext:master from anktx:patch-1

anktx commented Oct 12, 2013

No description provided.

andrey82k added 2 commits

October 12, 2013 13:04


          do not rewrite existing slug

ec83a14


          generate unique slug by appending index to existing slug

33b7462

Member

slavcodev commented Oct 12, 2013

I do not think that this commit will be merged.

For don't rewrite slug attach behavior only in insert scenario.
For unique slug use id or date prefix but in other behavior or in before validate, the slug strategy is not responsibility of this extension.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet