Pending

User tests: Successful: Unsuccessful:

avatar Paladin
Paladin
30 Mar 2013

JTable::getInstance quit when the requested file was not found in the first file name matched, even though there may be more file names that match and which might contain the right class. This change keeps it going until it's checked every matching file

This one we may want to talk about..there may be a better place and a better way to fix it. What happens is if you have two files that fit the name, but the first one doesn't have the right class, JTable quits without ever seeing the second one. I can see a discussion, though, about whether we want to enforce good naming conventions. It's an ideal vs real discussion.

avatar Paladin Paladin - open - 30 Mar 2013
avatar nicksavov
nicksavov - comment - 19 Apr 2013

Thanks for coding this, Arlen!

While we’re transitioning to a new integrated tracker, could you report the issue on our current main tracker at JoomlaCode and cross-reference each with a link to the other? Here’s the process for reporting on the other tracker:
http://docs.joomla.org/Filing_bugs_and_issues

Alternatively, let me know if you’d like me to create it for you and I can go ahead and do that.

Thanks in advance and thanks again for coding this, Arlen!

avatar phproberto
phproberto - comment - 13 Jun 2014

I think I have faced the same problem on J3.X:

https://github.com/joomla/joomla-cms/blob/staging/libraries/joomla/table/table.php#L266

A third 3rd part extension was adding a table path that included the same file name than my extension. The problem here is that is hard to test. I'll try to provide a sample extension for it.

avatar wilsonge wilsonge - reference | - 26 Jul 14
avatar wilsonge
wilsonge - comment - 26 Jul 2014

test: Works as expected. 3.x forward port of this #3975

avatar dbhurley dbhurley - reference | - 27 Jul 14
avatar Bakual Bakual - reference | ca277d9 - 8 Aug 14
avatar Bakual Bakual - close - 8 Aug 2014
avatar Bakual
Bakual - comment - 8 Aug 2014

Merged into 2.5.x. Thanks!

avatar Sophist-UK Sophist-UK - reference | 06d905c - 7 Oct 14

Add a Comment

Login with GitHub to post a comment