?

User tests: Successful: Unsuccessful:

avatar julienV
julienV
25 Apr 2014

Basically, if there is a library prefix 'R' and another 'Red', it will try to find classes from Red library in R library, and fail, so the Red classes will never be autoloaded.
this is something that is fixed in 3.x version

avatar julienV julienV - open - 25 Apr 2014
avatar rvbgnu
rvbgnu - comment - 26 Jul 2014

@test
Joomla 2.5.25-dev

  • Installed redEVENT 2.5.b.5.7.2
    http://redcomponent.com/redcomponent/redevent
    tehn the Admin component gave this error:
    ATTENTION: redform.core.php not found. A compatible version of redFORM component is MANDATORY for redEVENT to work properly.

  • Installed redCORE 1.20 from here
    https://github.com/redCOMPONENT-COM/redCORE/releases
    but could not complete installation:
    500 - An error has occurred.
    JDatabaseMySQLi::query: 1025 - Error on rename of './joomla_2@002e5_jbs_brochuresample/l8ex3_banner_clients_rctranslations' to './joomla_2@002e5_jbs_brochuresample/#sql2-1ed-7e' (errno: 152) SQL=ALTER TABLE l8ex3_banner_clients_rctranslations DROP FOREIGN KEY d5f871880cbf2de8c8b8a945c01f2e56

avatar rvbgnu
rvbgnu - comment - 26 Jul 2014

Tracker item Moved to Needs Review

avatar julienV
julienV - comment - 26 Jul 2014

well, the message says you need redFORM, not redCORE ;)

avatar rvbgnu
rvbgnu - comment - 26 Jul 2014

So do we need redCORE as well to test this bug, as explained in the test Instructions?
Moving the tracker to Information Required by the way ;-)

avatar julienV
julienV - comment - 31 Jul 2014

Hi,
i discussed with redCORE team, it appears that it's just a one time upgrade bug due to an older preversion of redCORE. You can actually install it by turning debug off....

avatar Bakual
Bakual - comment - 31 Jul 2014

So this can be closed?

avatar julienV
julienV - comment - 31 Jul 2014

no it means it's possible to test it... However, it's the same code as on 3.x, and if it wouldn't work, nothing would load at all.

avatar julienV
julienV - comment - 31 Jul 2014

ok guys, getting serious, i updated the PR with loader unit test, that tries to use 2 libraries sharing same prefix start (Dummy / Dummya).

You can run the test with the modified loader.php, and try it again with current (previous) loader, you will see the difference

avatar rvbgnu
rvbgnu - comment - 31 Jul 2014

Perfect timing, I've just downloaded redForm, and was going to test again ;-)

avatar rvbgnu
rvbgnu - comment - 31 Jul 2014

Thanks @julienV ! Does it mean that we do not need to install redComponents? Please provide new test instructions, I am moving on another issue in the meantime ;-)

avatar julienV
julienV - comment - 31 Jul 2014

you can test with redform if you like, but the unit test should in fact cover it. Just run it with and without the modification.

avatar rvbgnu
rvbgnu - comment - 31 Jul 2014

sorry you lost me there... How do I test without the modification (to reproduce the problem)? And how with the modif?

avatar julienV
julienV - comment - 1 Aug 2014

first, you can see that the modification works without regression by just having joomla properly displaying, as if the library wouldn't load, it plain wouldn't work.

Then, for the specific bug, it's verified by the unit test:
https://github.com/julienV/joomla-cms/blob/fix-autoloader/tests/unit/suite/libraries/loader.php
but of course, it requires that you are set to run unit tests....

you can also test by installing redcore, redform, and redevent, then browse the backend pages of redevent and redform.

avatar rvbgnu
rvbgnu - comment - 1 Aug 2014

Sorry, I am not set to run unit tests, yet. Please could you ping two person that are set to run unit tests, so that we could test and fixed this issue in JoomlaCode? There is only 17 issues open, including 3 RTC and 3 Pending...

avatar julienV
julienV - comment - 1 Aug 2014

i will ask people on redweb...

avatar Kixo
Kixo - comment - 11 Aug 2014

tested... works fine

avatar jatitoam
jatitoam - comment - 11 Aug 2014

Tested, it works ok

avatar rvbgnu
rvbgnu - comment - 12 Aug 2014

Thanks guys @Kixo @jatitoam for testing.
As we have two successful tests, moving to RTC.

avatar phproberto
phproberto - comment - 13 Aug 2014

Merged in 204a284.

Thanks!

avatar phproberto phproberto - change - 13 Aug 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-08-13 01:01:32
avatar phproberto phproberto - close - 13 Aug 2014

Add a Comment

Login with GitHub to post a comment