Skip to content

1.13.1 (and earlier): PUM_Telemetry triggers PHP warnings in (CLI) cron #919

@lkraav

Description

@lkraav

Describe the bug

Periodically cron spams us:

Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 139
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 142
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 139
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 140
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 139
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 140
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 139
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 140
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 139
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 140
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 139
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 140
Notice: Undefined index: cookies in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 105
Warning: Invalid argument supplied for foreach() in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 105
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 139
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 140
Notice: Undefined index: cookies in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 105
Warning: Invalid argument supplied for foreach() in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 105
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 139
Notice: Undefined index: open_sound in /home/convers/public_html/wp-content/plugins/popup-maker/classes/Telemetry.php on line 140

Site information

Popup Maker version: 1.13.1

WordPress version: 5.6

PHP version: 7.4

Expected behavior

Telemetry class should improve cron compatibility.

Current behavior

Probably some variables are not prepared for cron context.

Steps to reproduce

  1. Execute cron via wp --quiet cron event run --due-now

Errors

See bug description.

Additional context

Tracebacks

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions