{"issue":{"pull_request":{"diff_url":"https://github.com/joomla/joomla-platform/pull/397.diff","html_url":"https://github.com/joomla/joomla-platform/pull/397","patch_url":"https://github.com/joomla/joomla-platform/pull/397.patch"},"state":"open","user":{"avatar_url":"https://secure.gravatar.com/avatar/8f3be2c1d014f124f134c6b32e38b123?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"8f3be2c1d014f124f134c6b32e38b123","url":"https://api.github.com/users/chdemko","login":"chdemko","id":834574},"milestone":null,"labels":[],"assignee":null,"updated_at":"2011-11-10T12:47:30Z","closed_at":null,"comments":18,"url":"https://api.github.com/repos/joomla/joomla-platform/issues/397","body":"See\r\nhttp://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=25835\r\nhttp://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=26628\r\nhttps://github.com/joomla/joomla-platform/issues/227\r\n\r\nAs was done for controllers when searching their models using their getModel method, the idea behind this patch consists to add several things:\r\n\r\nJTable::addIncludePath\r\n\r\n  1. A call to JTable::addIncludePath() will return all paths previously added or the core tables folder of Joomla! (as it is before patch)\r\n  2. A call to JTable::addIncludePath('my/new/path') will add the new path to the global list and will return the global list (as it is before patch)\r\n  3. A call to JTable::addIncludePath('my/new/path', 'MyPrefixTable') will add the new path to the global list and to the 'MyPrefixTable' list and will return it\r\n\r\nJTable::getInstance\r\n\r\n  1. Search using the prefix specific list using JTable::addIncludePath(null, $prefix)\r\n  2. Fallback to search in the global list of JTable::addIncludePath()\r\n\r\nJModel::addTablePath\r\n\r\n  have an optional second parameter to be passed to JTable::addIncludepath\r\n\r\nJModel::__construct\r\n\r\n  1. can defined a table_prefix value\r\n  2. calls self::addTablePath using table_prefix as second parameter\r\n\r\n","html_url":"https://github.com/joomla/joomla-platform/issues/397","title":"Joomla CMS [#25835, #26628] JTable:getInstance doesn't manage correctly multiple includePaths","number":397,"id":1753592,"created_at":"2011-09-27T16:04:29Z"},"action":"created","comment":{"user":{"avatar_url":"https://secure.gravatar.com/avatar/679908c699f06435eb3c407f0259bab1?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"679908c699f06435eb3c407f0259bab1","url":"https://api.github.com/users/joomla-jenkins","login":"joomla-jenkins","id":929228},"updated_at":"2011-11-10T12:47:30Z","url":"https://api.github.com/repos/joomla/joomla-platform/issues/comments/2694770","body":"Build triggered by changes to the base.\n\nTest log missing. Tests failed to execute.\nCheckstyle analysis reported 233 warnings and 4165 errors.\n","id":2694770,"created_at":"2011-11-10T12:47:30Z"}}