Problem/Motivation

Notice: Undefined offset: 0 in Drupal\opigno_module\Form\OpignoActivityForm->buildForm() (line 104 of modules/contrib/opigno_module/src/Form/OpignoActivityForm.php

Notice: Trying to get property 'tid' of non-object in Drupal\opigno_module\Form\OpignoActivityForm->buildForm() (line 104 of modules/contrib/opigno_module/src/Form/OpignoActivityForm.php).

This happens if the opigno skills taxonomy is empty.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#3 opigno_module-3085194-3.patch1.17 KBcatch
#2 opigno_module-3085194.patch569 bytescatch

Comments

catch created an issue. See original summary.

catch’s picture

catch
he/him
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new opigno_module-3085194.patch569 bytes
catch’s picture

catch
he/him
Primary language English
commented
StatusFileSize
new opigno_module-3085194-3.patch1.17 KB

That patch causes a notice since it can leave $default_target_skill unset, better version.

plach’s picture

plach
he/him
Primary language Italian
Location Venezia
commented
Status: Needs review » Reviewed & tested by the community

Looks good

axelm’s picture

axelm
Primary language English
commented
Status: Reviewed & tested by the community » Patch (to be ported)

Hi
Thanks for reporting this issue.
The patch will be integrated to the next release of Opigno.

axelm’s picture

axelm
Primary language English
commented
Status: Patch (to be ported) » Fixed
axelm’s picture

axelm
Primary language English
commented
Status: Fixed » Closed (fixed)