?
avatar photodude
photodude
15 May 2017

Steps to reproduce the issue

random failure occurring in cachelite test usually with php 7.1 on windows
https://ci.appveyor.com/project/rdeutz/joomla-cms/build/1.0.4240/job/r3ms19arudrf3rtw

Expected result

no failure

Actual result

There was 1 failure:
1) JCacheStorageCacheliteTest::testCacheTimeout
Some data should be available in lifetime.
Failed asserting that false matches expected 'testData'.
tests\unit\core\case\cache.php:121
tests\unit\suites\libraries\joomla\cache\storage\JCacheStorageCacheliteTest.php:66

System information (as much as possible)

php 7.1 windows (appveyor)

//cc @mbabker

avatar photodude photodude - open - 15 May 2017
avatar joomla-cms-bot joomla-cms-bot - change - 15 May 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 15 May 2017
avatar joomla-cms-bot joomla-cms-bot - edited - 15 May 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 16 May 2017
Category Cache
avatar mbabker
mbabker - comment - 1 Jun 2017

Following the question everything mentality that seems to be recently popular, I'm wondering what value there is to still support Cache_Lite in 4.0. It's in essence an alternative to our filesystem cache adapter, and since we don't even ship it with core (we could thanks to Composer), it's only available on servers where the PEAR package is installed.

avatar photodude
photodude - comment - 1 Jun 2017

I see no reason to support it. I have never seen anyone actually using it.

avatar franz-wohlkoenig franz-wohlkoenig - change - 4 Jun 2017
Status New Discussion
avatar mbabker
mbabker - comment - 7 Jun 2017

Would you mind doing a PR for 3.8 to deprecate the Cache_Lite cache adapter? Then we can drop support in 4.0. If there are any issues, we can discuss them in the deprecation PR and issues like this can either be addressed at low priority or closed in full.

avatar brianteeman
brianteeman - comment - 7 Jun 2017

see #16575

avatar photodude
photodude - comment - 7 Jun 2017

closing via PR #16575

avatar photodude photodude - change - 7 Jun 2017
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2017-06-07 18:05:32
Closed_By photodude
avatar photodude photodude - close - 7 Jun 2017

Add a Comment

Login with GitHub to post a comment