Subject: downloads plugin from geeklog.fr breaks with geek log 2.1

Posted on: 29/07/14 01:30pm
By: remy

The downloads plugin 1.1 from geeklog.fr happens to use a gltext class, which does perfectly co-exist in geeklog 2.0.
Somehow geeklog 2.1 now loads by default a different gltext class, so the downloads plugin breaks with error "cannot declare GLText twice". Besides the plugin/gltext.class is quite different from system/classes/gltext.class.

Usage in geeklog2.1 is in lib-common in COM_renderWikiText and COM_checkHTML

So, what to obsolete now? lib-common or the downloads plugin?
What *standard* is followed now?

Re: downloads plugin from geeklog.fr breaks with geek log 2.1

Posted on: 29/07/14 03:14pm
By: Laugh

Dengen introduced the new gltext class to Geeklog 2.1.0. It now handles the rendering of the articles and comments.

Dengen is also the author of the Downloads plugin. I believe he is working on an update (hopefully he will see this post and add his comment) which will I assume use the newer Geeklog gltext class.

As a work around you could rename all instances of the gltext class in the downloads plugin to something else.

I will send him a quick email and get him to respond.

Re: downloads plugin from geeklog.fr breaks with geek log 2.1

Posted on: 30/07/14 08:46am
By: ::Ben

You can download a working version for Downloads plugin here. Warning this is a beta provide by Ivy.

Ben

Re: downloads plugin from geeklog.fr breaks with geek log 2.1

Posted on: 30/07/14 09:08am
By: remy

Thanks Ben, this one works with 2 tiny corrections:

The constructor method in include/gltext.class.php should be named to the className i.e. GLTextd()

The visit.php module uses the retrieved url as opposed to the combination of secret_filename

Than, the plugin lacks the downloads folder structures, which is not a problem since they are in place.

I took the plugin without visit.php (used the original) and corrected the class constructor method name.
Works. Again, thanks.

Geeklog - Forum
https://www.geeklog.net/forum/viewtopic.php?showtopic=95746