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

[question] standalone 5.7 version or how to update for e107 CMS #19

Closed
Jimmi08 opened this issue Jul 8, 2020 · 4 comments
Closed

[question] standalone 5.7 version or how to update for e107 CMS #19

Jimmi08 opened this issue Jul 8, 2020 · 4 comments

Comments

@Jimmi08
Copy link
Contributor

Jimmi08 commented Jul 8, 2020

Hello,
at first, thank you for all your job with maintaining this.

I was asked to update e107 plugin by the same author (only e107 stuff, not LGSL). But the last version is for LGSL 5.7 and there were changes against 5.8.

I would like to use 5.10.3 directly, but I don't know what are differences against SA version in lgsl code. (e107 stuff is no problem)

For now I found only $lgsl_config['cms']

I thought if I compare LGSL 5.7 SA vs e107 lgsl_files, that I could check this.

Can somebody share this version? Or confirm that that cms config is the only difference?

Thank you

@tltneon
Copy link
Owner

tltneon commented Jul 8, 2020

Hello! I'm glad a lot by your words.

To upgrade lgsl 5.7 you should backup your current files and to replace with this ones: lgsl_class.php, lgsl_protocol.php, lgsl_admin.php
Also add new folder languages, styles and scripts. And in the end you need to add a few new lines in lgsl_config.php

Honestly, I didn't do that but LGSL 5.10.3 mostly should be compatible with older version.

@Jimmi08
Copy link
Contributor Author

Jimmi08 commented Jul 9, 2020

@tltneon
You are right and I decided on a different approach. I use directly 5.10.3 version, no upgrade process and I created (in progress) new e107 files. For now, everything works and it is clear. In testing phase now.
I have issues only with userbar.php (but it doesn't work with SA version either), but it can be set it off. No priority.

tltneon added a commit that referenced this issue Jul 9, 2020
change related to new e107 version #19
@Jimmi08
Copy link
Contributor Author

Jimmi08 commented Jul 10, 2020

@tltneon
great work really, it worked out of the box. Without you, this plugin will be lost.

I will publish code after they will end with testing. They can compare with SA version, so they know if it worked before or not.

image

I just added config to the admin area to avoid manual changing config file (they have 12 zones)

image

image

Frontend with darken style:

image

@tltneon
Copy link
Owner

tltneon commented Sep 8, 2020

I almost forgot to say you that you made awesome e107 plugin! Good job

@tltneon tltneon closed this as completed Sep 8, 2020
@TacTicTow TacTicTow mentioned this issue Mar 4, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants