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

Logbook generator doesn't work. Code refactoring required #655

Open
mzylowski opened this issue May 24, 2016 · 6 comments
Open

Logbook generator doesn't work. Code refactoring required #655

mzylowski opened this issue May 24, 2016 · 6 comments

Comments

@mzylowski
Copy link
Member

mzylowski commented May 24, 2016

http://opencaching.pl/logbook.php?logbook_type=2 :
selection_031

We didn't change anything in code related with this, so it is probably problem with configuration or something else.

In oc.nl form shows up correctly:
selection_032

@deg-pl It is probably post-migration problem so you are pointed here :)

@harrieklomp
Copy link
Member

Perhaps this has to do with translations?
When change site language to Dutch i get a white screen on oc.nl. All other languages are working correct.

@deg-pl
Copy link
Member

deg-pl commented May 24, 2016

No. It's a deeper problem. I'm working on fix.
After:

apt install php5.6-mcrypt
service apache2 restart

is a little better, but still is problem ;)
BTW: On OC.NL also did not work correctly:
logbook1
logbook2

@deg-pl
Copy link
Member

deg-pl commented May 24, 2016

logbook.php & /logbook/*.php - this is a VERY ARCHAIC code! It has a lot of errors. This code requires a complete rewrite from scratch. In this form it does not appear to work for many years and is not related to migration (exept absence of php5.6-mcrypt). Uses /logbook/work as temorary space (not default dynamic-files) - we haven't this directory for many years!

We have NO link to this files in all OC code. Only link is at Wiki (https://wiki.opencaching.pl/index.php/U%C5%BCyteczne_pliki_zwi%C4%85zane_z_OC_PL)

My suggestion is to remove this code and link from the Wiki.

@deg-pl deg-pl removed their assignment May 24, 2016
@mzylowski
Copy link
Member Author

mzylowski commented May 24, 2016

Actually I know few people who uses it. (You think how I know about these problems? ;])
Also as I know it is only one way to easy create logbook with our date etc.
It is important feature, especially for new or inexperienced users.

If we need refactoring we should just do it ;]

Edit: It is on my now, but if somebody want to work on it -> fell free ;]

@mzylowski mzylowski self-assigned this May 24, 2016
@deg-pl
Copy link
Member

deg-pl commented May 24, 2016

We have a number of very important issues unresolved. Please - resolve first #645, #628, #618, #611, #605, #561

@mzylowski mzylowski changed the title Logbook generator doesn't work on OC.pl (after migration) Logbook generator doesn't work. Code refactoring required May 25, 2016
@kojoty kojoty mentioned this issue Jul 19, 2017
21 tasks
@deg-pl
Copy link
Member

deg-pl commented Aug 20, 2018

mcrypt is deprecated in PHP 7.1 and dropped in PHP 7.2
We use mcrypt only in logbook generator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants