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

Create table, schema reference fix #3213

Closed
wants to merge 4 commits into from
Closed

Create table, schema reference fix #3213

wants to merge 4 commits into from

Conversation

KoriSeng
Copy link
Contributor

No description provided.

Made getBestLanguage to return family language by default.

I believe most of the developers already manually substr the entry for
translation. This should reduce that 1/2 lines
@maxgalbu
Copy link
Contributor

you are mixing two fixes (a fix for mysql and a fix for #3135) in the same pull request. You should remove the last two commits from this branch, then create another branch with the fixes for #3135 and make another pull request.

Also, this pull request is on master, which is not accepted by phalcon:

#3204

Taking advantage of keep snapshot,

It will attempt to save parent on preSave,
Check itself for changes, if no changes just attempt to save child.

Note. PostSave for the specific model will not be called.

$user = User::findFirst(array("id = 🆔",  "bind" => array("id" =>
1)));
$user->save(); <<== not called
$user = User::findFirst(array("id = 🆔",  "bind" => array("id" =>
2)));
$user->primary_email = "test@test.com";
$av = new AccountVerify();
$av->key = "asdasdsad"; // processed as child
$user->accountverify = $av;
$user->save(); <<== called
$user = User::findFirst(array("id = 🆔",  "bind" => array("id" =>
3)));
$user->save(); //not called
@KoriSeng
Copy link
Contributor Author

I aint good with this git, been sometime since i used it. confusing as shit. i am trying to reconcile all changes back to 2.0.0

@KoriSeng KoriSeng closed this Jan 12, 2015
@KoriSeng KoriSeng deleted the patch-2 branch January 12, 2015 18:53
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