03/06/17 12:38am  

Harold

Anonymous
I created a Link and now I am unable to edit the link. When I click on the pencil, it opens the page for a new link.

This is on version 2.1.2.

 03/06/17 06:53am  
Status: offline

Laugh

Site Admin
Admin
Registered: 09/27/05
Posts: 1468
Thanks for the report. We hope to have it fixed soon.

For more details see the issue on GitHub: https://github.com/Geeklog-Plugins/forum/issues/21
One of the Geeklog Core Developers.
 05/08/17 05:33pm  
Status: offline

BruceG

Forum User
Newbie
Registered: 05/08/17
Posts: 1
The bbcode bug for buttons is fixed in the latest version at least, but this bug persists. I fixed it locally by changing line 531 of admin/plugin/links/index.php from:

Text Formatted Code
$display .= editlink($mode, Geeklog\Input::fGet('id'));
to
Text Formatted Code
$display .= editlink($mode, $_GET['lid']);
 05/24/17 03:14am  

Harold

Anonymous
Quote by: BruceG

The bbcode bug for buttons is fixed in the latest version at least, but this bug persists. I fixed it locally by changing line 531 of admin/plugin/links/index.php from:

Text Formatted Code
$display .= editlink($mode, GeeklogInput::fGet('id'));
to
Text Formatted Code
$display .= editlink($mode, $_GET['lid']);



Thank you for the information, Bruce!

I had been unable to decipher the issue and the github recommendation did nothing to clarify the true issue. I ended up abandoning the use of Geeklog. I really enjoyed using it for awhile.

 06/29/17 09:07pm  

Harold

Anonymous
I downloaded v2.1.3 to try it out again.

The issue with being able to edit weblinks is still there.

Your code modification did the trick. Once again, thanks!!
 06/30/17 07:00am  
Status: offline

Laugh

Site Admin
Admin
Registered: 09/27/05
Posts: 1468
Thanks for the report. This issue somehow got missed. I created a new separate issue for it on GitHub so it doesn't get missed again. Cool

https://github.com/Geeklog-Core/geeklog/issues/798
One of the Geeklog Core Developers.
 06/30/17 09:53am  
Status: offline

masodo

Forum User
Chatty
Registered: 11/13/12
Posts: 35
Thanks Harold - you beat me to it Big Grin
Thank you BruceG for the quick-fix... Works like a charm! Cool
 09/20/17 10:24am  
Status: offline

Laugh

Site Admin
Admin
Registered: 09/27/05
Posts: 1468
This issue is now fixed in the repository on GitHub. The correct fix is to replace the above line with:

Text Formatted Code

$display .= editlink($mode, Geeklog\Input::fGet('lid'));
 


The only thing wrong was the name of the variable. The other fix works but is not as secure.
One of the Geeklog Core Developers.