Welcome to Geeklog, Anonymous Saturday, December 21 2024 @ 11:08 am EST
Geeklog Forums
downloads plugin from geeklog.fr breaks with geek log 2.1
Status: offline
remy
Forum User
Full Member
Registered: 06/09/03
Posts: 162
Location:Rotterdam & Bonn
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?
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?
13
11
Quote
Status: offline
Laugh
Site Admin
Admin
Registered: 09/27/05
Posts: 1470
Location:Canada
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.
One of the Geeklog Core Developers.
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.
One of the Geeklog Core Developers.
15
13
Quote
Status: offline
remy
Forum User
Full Member
Registered: 06/09/03
Posts: 162
Location:Rotterdam & Bonn
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.
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.
12
9
Quote
All times are EST. The time is now 11:08 am.
- Normal Topic
- Sticky Topic
- Locked Topic
- New Post
- Sticky Topic W/ New Post
- Locked Topic W/ New Post
- View Anonymous Posts
- Able to post
- Filtered HTML Allowed
- Censored Content