Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

xExtension-SendToMyJD2: Undefined variable: $lMD #30

Closed
Alkarex opened this issue Aug 18, 2024 · 4 comments
Closed

xExtension-SendToMyJD2: Undefined variable: $lMD #30

Alkarex opened this issue Aug 18, 2024 · 4 comments
Labels
no-issue-activity

Comments

@Alkarex
Copy link

Alkarex commented Aug 18, 2024 •
edited
Loading

A little bug: you cannot use a variable after unset()

cntools_FreshRssExtensions/xExtension-SendToMyJD2/extension.php

Lines 304 to 305 in d0bd817

unset($lMD);
$this->cntMyJdMD = $lMD;

By the way, you are welcome to have a look at automatic tests in https://github.com/FreshRSS/Extensions for catching errors and warnings (FreshRSS/Extensions#248):

image
@github-actions GitHub Actions
Copy link

Welcome Alkarex 🎉

Congrats to your first issue!

@github-actions GitHub Actions
Copy link

Stale issue message

@Alkarex
Copy link
Author

Alkarex commented Oct 24, 2024

See example of code update for another extension: Joedmin/xExtension-wallabag-button#1

@Alkarex
Copy link
Author

Alkarex commented Jan 11, 2025

Still relevant. See update FreshRSS/Extensions#269

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
no-issue-activity
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Alkarex