OC 2.3.0.2 + Journal 2 + php 7.3 - Cannot declare class Mobile_Detect

M

magbor

New Member
#1
#1
It is used already by journal2_cache.php - require_once(DIR_SYSTEM . 'journal2/lib/Mobile_Detect.php');
so include_once(DIR_SYSTEM . 'library/Mobile_Detect/Mobile_Detect.php'); leads to error 500
The error remains even if all journal cache is off and cache is cleared (by journal menu).
Can this be solved?
(site is magbor.com)
 
M

magbor

New Member
#3
#3
serpent_driver said:
A 500 error can mean anything and unless you post the entire error, unfortunately we can't help you. At least post the entire error message.
Click to expand...
in error_log:
PHP Fatal error: Cannot declare class Mobile_Detect, because the name is already in use in /home/magborco/public_html/system/library/Mobile_Detect/Mobile_Detect.php on line 23
when enabling lscache - the result in front is error 500
 
serpent_driver

serpent_driver

Well-Known Member
#4
#4
You have already given yourself the answer to your issue. Journal 2 uses a class with the same name as in the LScache plugin and this is not allowed or causes the error. The functionality for detecting devices works differently in Journal than in the LScache plugin and regularly leads to conflicts. You can try temporarily disabling the "Separate View for Mobile Device" feature in the cache plugin, which will probably fix the conflict, but then the device detection in Journal will no longer work.
 
AndreyPopov

AndreyPopov

Well-Known Member
#5
#5
magbor said:
It is used already by journal2_cache.php - require_once(DIR_SYSTEM . 'journal2/lib/Mobile_Detect.php');
so include_once(DIR_SYSTEM . 'library/Mobile_Detect/Mobile_Detect.php'); leads to error 500
The error remains even if all journal cache is off and cache is cleared (by journal menu).
Can this be solved?
Click to expand...
Journal2 and LScache plugin use same Mobile_Detect library.

then delete or rename
/home/magborco/public_html/system/library/Mobile_Detect/Mobile_Detect.php

or in file
/catalog/controller/extension/module/lscache.php

comment
PHP: 
// include_once(DIR_SYSTEM . 'library/Mobile_Detect/Mobile_Detect.php');
 
Last edited:
M

magbor

New Member
#6
#6
AndreyPopov said:
Journal2 and LScache plugin use same Mobile_Detect library.

then delete or rename
/home/magborco/public_html/system/library/Mobile_Detect/Mobile_Detect.php

or in file
/catalog/controller/extension/module/lscache.php

comment
PHP: 
// include_once(DIR_SYSTEM . 'library/Mobile_Detect/Mobile_Detect.php');
Click to expand...
Thank you very much for answering the question!
I tried first your second idea and commented the "include_once" line in lscache.php but this leads to error 500 again. If you are interested in the error I can post it.
But your first proposal works for now without any errors: before installing from the Modules menu I deleted the file /system/library/Mobile_Detect/Mobile_Detect.php and in headers, lscache is presented. www.magbor.com
The site is not faster at all for now, but I hope to find ways to make it better. The main thing is that it is fully functional and has no problems.
I fear only that the Mobile_Detect.php which Journal2 uses is an older version @version 2.8.25 than the one in lscache @version 2.8.34. Is that a problem? Thank you again for your efforts.
 
AndreyPopov

AndreyPopov

Well-Known Member
#7
#7
magbor said:
I fear only that the Mobile_Detect.php which Journal2 uses is an older version @version 2.8.25 than the one in lscache @version 2.8.34. Is that a problem? Thank you again for your efforts.
Click to expand...
not fear. no problem.

Journal decide what mobile or no. lscache only cache.
if Journal says - mobile, lscache mark copy of page as mobile
 
M

magbor

New Member
#8
#8
AndreyPopov said:
not fear. no problem.

Journal decide what mobile or no. lscache only cache.
if Journal says - mobile, lscache mark copy of page as mobile
Click to expand...
I see I will need a lot of help to get it working right.. :) Too many questions -

do I need to uncomment a separate mobile view... and so on

Thank you for your cooperation, I appreciate it much! If I am sure what to ask, I will place a question again
 
serpent_driver

serpent_driver

Well-Known Member
#10
#10
@magbor
Unfortunately, what @AndreyPopov says is not entirely true. The LScache plugin not only caches. The cache varies (rewrite rules in .htaccess) are the counterpart to the PHP class Mobile_Detect. If you need device detection for Mobile View, then these Cache Varies in .htaccess are mandatory. Otherwise the cache plugin does not know what a mobile device is and the display will be incorrect.
 
AndreyPopov

AndreyPopov

Well-Known Member
#11
#11
serpent_driver said:
@magbor
The LScache plugin not only caches.
Click to expand...
really? are you sure?



serpent_driver said:
@magbor
If you need device detection for Mobile View, then these Cache Varies in .htaccess are mandatory. Otherwise the cache plugin does not know what a mobile device is and the display will be incorrect.
Click to expand...
LScache use COOKIES for this and .htaccess rewrite rules only nessesary for Bots and devices (UAs) that not accept cookies!
 
You must log in or register to reply here.
Top