BUGFIX: Removed assumption in GridFieldDataColumnsTest that Member would... #354

Merged
merged 1 commit into from Apr 19, 2012

Conversation

Projects
None yet
2 participants
Contributor

robert-h-curry commented Apr 19, 2012

... always have the same summary fields.

Was causing a problem when Member was customised.

Should I take the fix further remove the use of 'Member', and instead use a custom class just for the test?

Owner

halkyon commented Apr 19, 2012

Probably the best we can do, otherwise it breaks with custom extensions to Member. Thanks Robert!

halkyon merged commit ee7d2c3 into silverstripe:master Apr 19, 2012

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