Ticket #29 (closed Bug: fixed)

Opened 17 months ago

Last modified 17 months ago

CI_Loader::database() assumes $CI->db is in use after a returned DB load

Reported by: JAAulde Owned by: JAAulde
Priority: major Milestone: 1.0
Component: Version:
Keywords: Cc:

Description

reported by me here: http://codeigniter.com/bug_tracker/bug/1813/

I have a fix for this which will be best implemented after ticket #28 is fixed because of the confusion with the active record flag which I complained about in the comments of the bug report at CI.

Change History

Changed 17 months ago by JAAulde

  • owner set to JAAulde

Changed 17 months ago by JAAulde

I adjusted the conditionals in CI_Loader:database() to check for presence of $CI->db rather than instantiation of CI_DB when making decision to return early without connecting database request.

Committed this to trunk but could not merge it to 1.0--opened ticket #32.

I tested these changes and it works well. This ticket can be closed after the merge to 1.0.

Changed 17 months ago by JAAulde

  • status changed from new to closed
  • resolution set to fixed

Closing even though #32 is not done. Clarification received in IRC made me more comfortable.

Note: See TracTickets for help on using tickets.