The module throws an error.
Warning: Undefined variable $websites in Drupal\do_username\DOComputedFields->getValue() (line 67 of /app/web/modules/contrib/do_username/src/DOComputedFields.php).
Enter a user who has no websites listed on their Drupal.org profile and try to read all the fields. This is currently only possible via code but it can happen when we try to serialize the field.
It's a simple fix. The variable `$websites` should be reset to an empty array on line 64 in the file. This likely hides other bugs as well but the fix is simple.
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | Undefined-variable-websites-3394978-1.patch | 419 bytes | mohd sahzad |
Show commandsStart within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes, plain diff MR !7
Comments
Comment #2
hussainwebComment #3
mohd sahzad commentedComment #4
mohd sahzad commentedI have updated the issue regarding the "Undefined variable $websites in Drupal\do_username\DOComputedFields->getValue()." please review the attached patch for the fix.
Comment #5
mohd sahzad commentedComment #9
hussainwebPatch in #4 is good. I am afraid that MR !7 is targeting the wrong branch.
Comment #11
hussainwebCommitted, thanks!