Skip to content

Use isset() to avoid warning - #653

Draft
leedxw wants to merge 3 commits into
developfrom
fix/warning
Draft

Use isset() to avoid warning#653
leedxw wants to merge 3 commits into
developfrom
fix/warning

Conversation

@leedxw

@leedxw leedxw commented Dec 11, 2024

Copy link
Copy Markdown
Contributor
PHP Warning:  Undefined property: Dxw\DxwSecurity2017\Lib\PluginVersionChecker::$slug in /var/www/html/wp-content/themes/dxw-security-2017/app/Lib/PluginVersionChecker.php on line 98

@leedxw
leedxw requested a review from snim2 December 11, 2024 16:54
@leedxw
leedxw requested a review from brent-dxw April 9, 2025 09:49
@snim2

snim2 commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

I think the code I've pushed here makes sense, but I can't replicate the warning with curl -I http://localhost/wp-json/v1/inspections/flibble

@snim2
snim2 marked this pull request as draft June 6, 2025 11:02
@snim2

snim2 commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

Also this branch will need to be rebased if we want to merge this

@leedxw

leedxw commented Apr 9, 2026

Copy link
Copy Markdown
Contributor Author
PHP Warning:  Undefined property: stdClass::$slug in /var/www/html/wp-content/themes/dxw-security-2017/app/Theme/Helpers.php on line 58
PHP message: PHP Warning:  Undefined property: stdClass::$name in /var/www/html/wp-content/themes/dxw-security-2017/app/Theme/Helpers.php on line 58

request: "GET /wp-admin/edit.php?post_type=plugins HTTP/1.1"
host: "advisories.dxw.com"
referrer: "https://advisories.dxw.com/wp-admin/post-new.php?post_type=plugins"

leedxw and others added 3 commits April 9, 2026 10:52
```
PHP Warning:  Undefined property: Dxw\DxwSecurity2017\Lib\PluginVersionChecker::$slug in /var/www/html/wp-content/themes/dxw-security-2017/app/Lib/PluginVersionChecker.php on line 98
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants