-
Notifications
You must be signed in to change notification settings - Fork 820
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
Add AddToCampaign dialog box functionality #5368
Add AddToCampaign dialog box functionality #5368
Conversation
Also see silverstripe/silverstripe-cms#1459 |
555bf04
to
a4437fa
Compare
* Add to campaign dialog | ||
* -------------------------------------------- */ | ||
|
||
#add-to-campaign__dialog { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We shouldn't be using ID selectors in CSS.
I'm going to stop reviewing and just push up phpdoc fixes. :D |
|
||
$fields = new FieldList([$campaignDropdown]); | ||
|
||
foreach ($this->data as $k => $v) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought you were going to change this? I'm still getting array to string conversion
when pushing data that is nested array content.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can change this to a recursive walk (probably in another method) that knows how to reconstruct a set of Name[Field] hiddenfields for each nested array.
7ca1e31
to
75b23e5
Compare
75b23e5
to
97e6727
Compare
No description provided.