Add support for many_many_extraField in YAML #2894

Merged
merged 1 commit into from Mar 3, 2014

Conversation

Projects
None yet
2 participants
@wilr
Owner

wilr commented Feb 24, 2014

Not sure if this is prefered syntax, I debated the 'named' syntax but this seems cleanest. Also we don't document the yaml syntax anywhere so couldn't find a good place to document this.

@simonwelsh

This comment has been minimized.

Show comment Hide comment
@wilr

This comment has been minimized.

Show comment Hide comment
@wilr

wilr Feb 24, 2014

Owner

Man search on doc.silverstripe.org is ass.. Thanks will document.

Owner

wilr commented Feb 24, 2014

Man search on doc.silverstripe.org is ass.. Thanks will document.

@simonwelsh

This comment has been minimized.

Show comment Hide comment
@simonwelsh

simonwelsh Feb 24, 2014

Contributor

Pull requests welcome :p

Contributor

simonwelsh commented Feb 24, 2014

Pull requests welcome :p

Add support for many_many_extraField in YAML
Format is

RelationName:
  - =>Obj.name:
    ExtraFieldName: "Foo"
@wilr

This comment has been minimized.

Show comment Hide comment
@wilr

wilr Feb 24, 2014

Owner

Thanks @simonwelsh for picking up that fixtures page, that's perfect. Have added a full example.

Owner

wilr commented Feb 24, 2014

Thanks @simonwelsh for picking up that fixtures page, that's perfect. Have added a full example.

simonwelsh added a commit that referenced this pull request Mar 3, 2014

Merge pull request #2894 from wilr/3.1-manymanyyaml
Add support for many_many_extraField in YAML

@simonwelsh simonwelsh merged commit ccb7919 into silverstripe:3.1 Mar 3, 2014

1 check passed

default Scrutinizer: No new issues — Travis: Passed
Details

@wilr wilr deleted the wilr:3.1-manymanyyaml branch Jul 27, 2014

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