The 2.8 changes to the OpignoModule::save() have broken the add/edit of the opigno_module entity when using the default entity add, edit admin pages (/admin/structure/opigno_module/add and /admin/structure/opigno_module/{opigno_module}/edit respectively).

line 312 of profiles/opigno_lms/modules/opigno/opigno_module/src/Form/OpignoModuleForm.php (Drupal\opigno_module\Form\OpignoModuleForm->save()) is trying to get the route parameter 'group' and it's subsequent id.

To recreate:
Login and go to - /admin/structure/opigno-modules
select to "add module"
complete and submit form

or

Login and go to - /admin/structure/opigno-modules
Select to edit from existing list of modules
make changes to form and submit

Comments

LittleCoding created an issue. See original summary.

LittleCoding’s picture

LittleCoding
He/Him
Location Hamilton, ON
CreditAttribution: LittleCoding commented

This is also effecting the duplicate option

axelm’s picture

axelm
Primary language English
CreditAttribution: axelm commented
Status: Active » Patch (to be ported)

Hi @LittleCoding
Thanks for reporting this issue.
There is indeed a regression in Opigno 2.8, it will be fixed in version 2.9

mdyoung3’s picture

mdyoung3 CreditAttribution: mdyoung3 commented

This seemed to be a problem for me with Opigno 2.9. After updating to 2.10, the issues was resolved.

LittleCoding’s picture

LittleCoding
He/Him
Location Hamilton, ON
CreditAttribution: LittleCoding commented
Status: Patch (to be ported) » Fixed

The fix was included as part of 2.10 so I am calling this one fixed!

Status: Fixed » Closed (fixed)

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