Closed (fixed)
Project:
Require on Publish
Version:
8.x-1.0-beta7
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Reporter:
Created:
23 Oct 2018 at 15:18 UTC
Updated:
6 Nov 2018 at 19:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
spfaffly commentedAttached patch. Thanks for a great module.
Comment #3
spfaffly commentedFound another instance of Undefined Index: Rerolled patch to include the fix.
Comment #4
spfaffly commentedComment #5
mpriscella commentedThanks, good catch! Changes look good, I added a small tweak to check the actual value of
#required_on_publishafter checking the existence of that key.Comment #6
spfaffly commentedLooks great. Totally makes sense to add the original logic back in 👍
Comment #7
mpriscella commentedAwesome, thanks! Merging this into the dev branch now.
Comment #9
mpriscella commented