? Pending

User tests: Successful: Unsuccessful:

avatar csthomas
csthomas
18 Feb 2017

Pull Request for errors on front end mentioned at #14102 (comment)

Summary of Changes

sqlsrv_fetch_object($cursor, $class) return null if Class from $class is not loaded.

To be sure that class is loaded I have added function class_exists($class) before calling sqlsrv_fetch_object.

Reason is not loaded class JMenuItem at:
https://github.com/joomla/joomla-cms/blob/staging/libraries/cms/menu/site.php#L88

Testing Instructions

Install latest jooma 3.7 on database SQLSRV on Windows platform.
Go to front page.

Expected result

Home page works.

Actual result

Component not found. Error 404

Documentation Changes Required

None

avatar csthomas csthomas - open - 18 Feb 2017
avatar csthomas csthomas - change - 18 Feb 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Feb 2017
Category MS SQL Libraries
avatar mbabker
mbabker - comment - 18 Feb 2017

I really wish we could stop having this discussion about why Microsoft's driver doesn't correctly load classes and they would fix their code...

avatar csthomas
csthomas - comment - 18 Feb 2017

Microsoft driver works correctly on linux:). On windows I do not know why, it has to be loaded manually.

Do you thing, this is the right place to do it or I should move it higher to loadObject and loadObjectList.

avatar alikon
alikon - comment - 19 Feb 2017

that's funny

Microsoft driver works correctly on linux:). On windows I do not know why, it has to be loaded manually.

avatar alikon alikon - test_item - 19 Feb 2017 - Tested successfully
avatar alikon
alikon - comment - 19 Feb 2017

I have tested this item successfully on 93e5203


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14137.

avatar waader waader - test_item - 19 Feb 2017 - Tested successfully
avatar waader
waader - comment - 19 Feb 2017

I have tested this item successfully on 93e5203


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14137.

avatar jeckodevelopment jeckodevelopment - change - 19 Feb 2017
Status Pending Ready to Commit
avatar jeckodevelopment
jeckodevelopment - comment - 19 Feb 2017

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/14137.

avatar wilsonge wilsonge - change - 19 Feb 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-02-19 19:17:22
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 19 Feb 2017
avatar wilsonge wilsonge - merge - 19 Feb 2017

Add a Comment

Login with GitHub to post a comment