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

ActiveRecord Interface and BaseAR #1359

Merged
merged 6 commits into from Nov 30, 2013
Merged

ActiveRecord Interface and BaseAR #1359

merged 6 commits into from Nov 30, 2013

Conversation

@cebe
Copy link
Member

cebe commented Nov 29, 2013

  • create AR Interface
  • create AR base class
  • fix problem with update()/insert()
  • update AR extensions to use Base AR
  • adjust docs in Base AR
  • fix redis AR tableName()

fixes #886, fixes #1205

cebe added 5 commits Nov 29, 2013
* master: (30 commits)
  replaced inheritdoc tags.
  doc cleanup.
  Moved key serialization from data provider to data viewers.
  Fixes #1356: Alias for authFile in rbac/PhpManager not resolved
  Fixes #1296: stricter check of dashes in route.
  Fixes #1362: added itemOptions to Html::radioList and checkboxList.
  Fixes #1362: doc fix.
  really small doc typo fix
  Fixes #1332: generate subdirectories when needed during message extraction process.
  typo fix.
  Fixes #1335: support view extension fallback
  More TDB for the guide
  Added reference to Composer section of the guide from 1.1.x differences page
  Better docs structure and minor fixes
  More docs on performance
  Structure for console application docs
  Docs on using behaviors
  doc fix.
  Refactored MessageInterface::send().
  refactored EmailTarget with the new mailer interfaces.
  ...
@cebe
Copy link
Member Author

cebe commented Nov 30, 2013

Forgot to re-add the tableName() to redis. will fix this tomorrow.

@cebe
Copy link
Member Author

cebe commented Nov 30, 2013

@qiangxue ready for merge now.

@ghost ghost assigned qiangxue Nov 30, 2013
qiangxue added a commit that referenced this pull request Nov 30, 2013
ActiveRecord Interface and BaseAR
@qiangxue qiangxue merged commit f5f443a into master Nov 30, 2013
1 check passed
1 check passed
default The Travis CI build passed
Details
@qiangxue qiangxue deleted the activerecord-interface branch Nov 30, 2013
@qiangxue
Copy link
Member

qiangxue commented Nov 30, 2013

Great job! Thanks!

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

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.