getParentEntity in SmartDateRule Preventing New Instance Creation

Thanks for identifying this, and for providing a fix. I updated the code to resolve the issues identified by the CI, and I refactored the new validateRule() method to make the logic easier to follow.

Please validate that the result still works for your use case.

Thank you for doing the code cleanup. I've created patches but this was my first MR.

It all looks good to me. I just made a couple of minor changes to the comments in the code to reflect the refactoring.

Great. Merged in, and should have a new release out shortly.

@esomething please don't open a new MR on an issue that's already merged. Your changes seem worthwhile so please open a new issue for them.

Sorry for that. Would you prefer a new MR or a new issue ticket?

If you could file a new issue that would be great. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.