We have an

    if (isset($flagging)) {

and an

    if (!isset($flagging)) {

in two separate places, and I don't think they need to be apart. The code would make more sense if they were together as an if/else block.

Patch coming.

CommentFileSizeAuthor
#1 2257603.flag_.flagging-entity-creation-cleanup.patch1.54 KBjoachim

Comments

joachim’s picture

joachim commented
Status: Active » Needs review
StatusFileSize
new 2257603.flag_.flagging-entity-creation-cleanup.patch1.54 KB

  • Commit 939b8e0 on 7.x-3.x by joachim: 
    Issue #2257603 by joachim: Moved code for populating or creating a...
joachim’s picture

joachim commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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