Skip to content
This repository was archived by the owner on Oct 29, 2020. It is now read-only.

Update status on campaign save#6120

Merged
sbsmith86 merged 5 commits intoDoSomethingArchive:devfrom
sbsmith86:update-status-campaign-save
Feb 3, 2016
Merged

Update status on campaign save#6120
sbsmith86 merged 5 commits intoDoSomethingArchive:devfrom
sbsmith86:update-status-campaign-save

Conversation

@sbsmith86
Copy link
Contributor

What's this PR do?

Updates the campaign status when someone changes the campaign run on campaign nodes.

Also, updated the hooks I was using to update the campaign status on campaign run node save to use hook_node_submit so we can more reliably get the values that had just been submitted by the user.

Where should the reviewer start?

Start at dosomething_campaign.module where I use hook_node_validate() to get the form state of the campaign node that is submitted and allows me to update the value of fields before the form actually gets submitted. This is where we update the status.

What are the relevant tickets?

Fixes #6052

Shae Smith added 4 commits February 3, 2016 09:53
…ission.

Use `hook_node_submit()` on campaign run submission to update the campaign node.
Use `hook_node_validate()` on campaign submission to update the campaign node.
@sbsmith86 sbsmith86 force-pushed the update-status-campaign-save branch from 8a8eb9e to 9242477 Compare February 3, 2016 14:56
@sbsmith86
Copy link
Contributor Author

@angaither updated the campaign status function to check for null dates.

@angaither
Copy link
Contributor

👍

sbsmith86 added a commit that referenced this pull request Feb 3, 2016
@sbsmith86 sbsmith86 merged commit c0f4a50 into DoSomethingArchive:dev Feb 3, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants