When cloning an entry with Ansel content, a PHP notice is displayed when EE Pro is enabled

#214 opened by Imp Designs

Subscribe

Description

To be clear, the clone works, it's just that these warnings display if there's content in an Ansel field. If there isn't any content in the Ansel field, everything's peachy.

I initially put in a support ticket at EE and, after confirming I was running Ansel 2.3.0, Robin suggested I drop you a note.

Notice

Undefined index: ansel_image_id

user/addons/ansel/Service/AnselImages/SaveRow.php, line 154

Severity: E_NOTICE
Warning

Cannot modify header information - headers already sent by (output started at ee/legacy/core/Exceptions.php:120)

ee/legacy/core/Input.php, line 266

Severity: E_WARNING
Warning

Cannot modify header information - headers already sent by (output started at ee/legacy/core/Exceptions.php:120)

ee/legacy/core/Input.php, line 266

Severity: E_WARNING
Warning

Cannot modify header information - headers already sent by (output started at ee/legacy/core/Exceptions.php:120)

ee/legacy/core/Input.php, line 266

Severity: E_WARNING
Warning

Cannot modify header information - headers already sent by (output started at ee/legacy/core/Exceptions.php:120)

ee/legacy/libraries/Functions.php, line 458

Severity: E_WARNING

Replies

  • TJ Draper

    Replied 6/21/2022 9:15 PM

    I have not been able to reproduce this issue in my testing with as close as I can get to the environment details you provided.

  • Imp Designs

    Replied 6/22/2022 8:46 AM

    Okay... well the client isn't freaking out so I guess it's not a big deal. Thanks for checking into it...

  • Imp Designs

    Replied 6/22/2022 10:42 AM

    DANG IT... I totally forgot a huge detail in this ticket...

    This is ONLY happening when EE Pro is enabled.

  • TJ Draper

    Replied 6/22/2022 10:44 AM

    That makes sense since cloning is an EE Pro only feature 🙂. I therefore did have EE Pro installed to test.

  • Imp Designs

    Replied 6/22/2022 11:52 AM

    Well then as Rosanne Rosannadanna said,

    never mind.

  • Oliver

    Replied 1/16/2023 8:05 AM, Edited 1/16/2023 8:05 AM

    I get the same warning testing with EE 7.2.6 and PHP 7.3.33, debugging enabled.

    Changing

    if ($data['ansel_image_id'])

    to

    if (isset($data['ansel_image_id']))

    in user/addons/ansel/Service/AnselImages/SaveRow.php, line 154 stops it.

  • TJ Draper

    Replied 4/22/2023 12:33 PM, Edited 4/22/2023 12:34 PM

    I was never able to replicate the error despite cranking error reporting all the way up dev. But the isset check doesn't hurt anything so I added it in Ansel 2.3.3

    https://www.buzzingpixel.com/software/ansel-ee/changelog/2-3-3

    https://www.buzzingpixel.com/software/ansel-ee/download

Use Markdown for formatting