- Issue created by @collinhaines
- Status changed to Needs review
almost 2 years ago 7:00pm 15 February 2023 - πΊπΈUnited States collinhaines
I found all instances of
new Instances();
and replaced it withClassResolverInterface::getInstanceFromDefinition()
.It appears that on the
SmartDateOverrideForm
, theform_builder
service was only being injected to be used as a constructor parameter. Since that was it's only usage and the controller initialization was replaced, I removed theform_builder
service from this form class. - π¨π¦Canada mandclu
This sounds a lot like an issue I encountered while testing the 4.0.x branch. That was documented and a fix committed at π Fatal Error: missing arguments when using remove instance form Closed: duplicate .
Your proposed code changes look similar to the changes committed in that issue. Can you validate if there's a difference or if that commit could be backported to 3.7.x?
-
mandclu β
committed fd5b3cd2 on 3.7.x
Issue #3342139 by collinhaines: Cannot remove recurring instance
-
mandclu β
committed fd5b3cd2 on 3.7.x
- π¨π¦Canada mandclu
I decided that the changes here are a better approach and more comprehensive than the changes in [#33341820], in addition to being written for 3.7.x. I have reverted the changes there and merged in your patch. Thanks for your work on this!
-
mandclu β
committed fd5b3cd2 on 4.0.x
Issue #3342139 by collinhaines: Cannot remove recurring instance
-
mandclu β
committed fd5b3cd2 on 4.0.x
-
mandclu β
committed 0b000f13 on 4.0.x
Issue #3342139 by collinhaines: Cannot remove recurring instance
-
mandclu β
committed 0b000f13 on 4.0.x
- Status changed to Fixed
almost 2 years ago 9:52pm 18 February 2023 Automatically closed - issue fixed for 2 weeks with no activity.