Hi there!

I've created a patch to add a simple separator (comma) in between items - if there is more than one item to display in a node.
I'd be glad if one of you could look at this and apply if ok.

Thanks, Marc

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

marcmueller created an issue. See original summary.

jalpesh’s picture

I am not able to apply your patch. Did you create a patch by using this https://www.drupal.org/node/707484 instruction.

marcmueller’s picture

I've made a diff on Version 7.x-1.1 in netbeans git utility, I'm not very familia using git directly.
Can I just upload 'entityreference.module' right here for you to make a patch out of this?

jalpesh’s picture

Please do that and also mention the changes in file.

Thanks

marcmueller’s picture

FileSize
43.97 KB

Thanks! Attached is the file.
1082: added default for separator setting
1116-1120: field formatter info
1161: summary display
1245-1258: output

Please note, as I can't upload as 'entityreference.module' I uploaded as 'entityreference.test' please rename accordingly.

Cheers, Marc

jalpesh’s picture

Assigned: Unassigned » jalpesh

Thanks marcmueller.

I am working on creating a patch and review you changes.

marcmueller’s picture

Thanks a lot!
Marc

jalpesh’s picture

Here you go with patch. I am working on verifying functionality with patch and update you soon.