- Issue created by @mparker17
- πΊπΈUnited States rclemings
I'm seeing the same thing. The watchdog notice appears to have been added by the patch in issue #3337784.
- πΊπΈUnited States DamienMcKenna NH, USA
That's.. frustrating. Because the whole point of #3337784 was to try to ensure the group description always existed :-\
- πΊπΈUnited States DamienMcKenna NH, USA
I reviewed the code a little more, and I think the log messages are unnecessary. So all we need to do is remove the two watchdog() calls.
- last update
over 1 year ago Build Successful - @mparker17 opened merge request.
- Status changed to Needs review
over 1 year ago 4:35pm 15 May 2023 - π¨π¦Canada mparker17 UTC-4
I've made a merge request; but please let me know if you'd prefer a patch instead. I couldn't find any tests for those watchdog lines, so I just deleted them.
@DamienMcKenna thank you very much for your quick response; and your patience.
-
DamienMcKenna β
committed 9f61e1e7 on 7.x-1.x authored by
mparker17 β
Issue #3360190 by mparker17, DamienMcKenna, rclemings: Unnecessary...
-
DamienMcKenna β
committed 9f61e1e7 on 7.x-1.x authored by
mparker17 β
- Status changed to Fixed
over 1 year ago 5:14pm 15 May 2023 -
DamienMcKenna β
committed c452a589 on 7.x-2.x authored by
mparker17 β
Issue #3360190 by mparker17, DamienMcKenna, rclemings: Unnecessary...
-
DamienMcKenna β
committed c452a589 on 7.x-2.x authored by
mparker17 β
Same issue and location is from the tracker:
The advanced tag group is missing a value for the description element.
The basic tag group is missing a value for the description element.
- πΊπΈUnited States DamienMcKenna NH, USA
@willj: Thank you for the report. The fix is currently in the dev version and will be in the upcoming 7.x-1.32 release.
Automatically closed - issue fixed for 2 weeks with no activity.