Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[BUG]xExtension-Copy2Clipboard - Fatal error for version number #6015

Closed
vdbhb59 opened this issue Jan 2, 2024 · 3 comments · Fixed by #6016
Closed

[BUG]xExtension-Copy2Clipboard - Fatal error for version number #6015

vdbhb59 opened this issue Jan 2, 2024 · 3 comments · Fixed by #6016
Assignees
Labels
Bug (confirmed) 🐞 issues that are reproducable Extension 🔌
Milestone

Comments

@vdbhb59
Copy link

vdbhb59 commented Jan 2, 2024

I am getting this error for xExtension-Copy2Clipboard:

Copy the destination links of all visible entries into clipboard
Fatal error: Uncaught TypeError: version_compare() expects parameter 2 to be string, float given in
/PATH/feeds/app/views/extension/index.phtml:64 Stack trace: #0
/PATH/feeds/app/views/extension/index.phtml(64): version_compare('0.4', 0.4) FreshRSS/Extensions#1
/PATH/feeds/lib/Minz/View.php(93): include('/home/USER1/...') FreshRSS/Extensions#2
/PATH/feeds/lib/Minz/View.php(115): Minz_View->includeFile('/views/extensio...') FreshRSS/Extensions#3
/PATH/feeds/app/layout/layout.phtml(73): Minz_View->render() FreshRSS/Extensions#4
/PATH/feeds/lib/Minz/View.php(93): include('/home/USER1...') FreshRSS/Extensions#5
/PATH/feeds/lib/Minz/View.php(106): Minz_View->includeFile('/layout/layout....') FreshRSS/Extensions#6
/PATH/feeds/lib/Minz/View.php(73): Minz_View->buildLayout() FreshRSS/Extensions#7
/PATH
/PATH/feeds/app/views/extension/index.phtml on line 64

Extension is downloaded from the provided one here: https://github.com/cn-tools/cntools_FreshRssExtensions/tree/master/xExtension-Copy2Clipboard

image

@Alkarex Alkarex transferred this issue from FreshRSS/Extensions Jan 2, 2024
@Alkarex Alkarex added this to the 1.24.0 milestone Jan 2, 2024
@Alkarex Alkarex self-assigned this Jan 2, 2024
Alkarex added a commit to Alkarex/FreshRSS that referenced this issue Jan 2, 2024
@Alkarex
Copy link
Member

Alkarex commented Jan 2, 2024

Thanks for the bug report.
Would you be able to test this patch? #6016
(Regression due to #5830)

@Alkarex Alkarex added the Bug (confirmed) 🐞 issues that are reproducable label Jan 2, 2024
@Alkarex Alkarex mentioned this issue Jan 2, 2024
2 tasks
@vdbhb59
Copy link
Author

vdbhb59 commented Jan 3, 2024

Thanks for the bug report. Would you be able to test this patch? #6016 (Regression due to #5830)

Anytime. Give me few hours. Will test and revert.

@vdbhb59
Copy link
Author

vdbhb59 commented Jan 3, 2024

Thanks for the bug report. Would you be able to test this patch? #6016 (Regression due to #5830)

Perfect solved the issue and also solved other installed + available extensions not showing up earlier.
Earlier under: Available community extensions
It would only show 4-5, now it shows all available as well as installed ones.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug (confirmed) 🐞 issues that are reproducable Extension 🔌
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants