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

BUG Renable the ability to do dynamic assignment with DBField #8821

Conversation

maxime-rainville
Copy link
Contributor

This partially restores the ability for DBField classes to do dynamic assignment if scalarValueOnly is false.

Parent issue

tests/model/ManyManyListTest.php Outdated Show resolved Hide resolved
tests/model/Mock/MockDynamicAssignmentDBField.php Outdated Show resolved Hide resolved
tests/model/Mock/MockDynamicAssignmentDBField.php Outdated Show resolved Hide resolved
tests/model/DataObjectTest.php Outdated Show resolved Hide resolved
@maxime-rainville
Copy link
Contributor Author

@robbieaverill All your points have been addressed.

@maxime-rainville
Copy link
Contributor Author

Actually SQLite and Postgress don't like my test. I'll have another go at it.

}

/**
* @expectedException PHPUnit_Framework_Error
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh yeah - user errors 😆

Copy link
Contributor

@robbieaverill robbieaverill left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still some super minor things like return types (e.g. mixed) not needing to be there, and double vs single quotes in the test dataobject, but nothing that blocks merge. Merge on green 👍

@robbieaverill robbieaverill merged commit e968454 into silverstripe:3.6 Feb 26, 2019
@maxime-rainville maxime-rainville deleted the pulls/3.6/restore-dynamic-field-assigment branch February 27, 2019 01:04
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.

None yet

2 participants