Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Superfluous behavior and non-DRY of calling classes through constructor too many times

#41

Each App_Model class has these same lines:

include(APPPATH.'config/databases.php');
$this->base = $this->load->database($db['base'], TRUE); $this->granary = $this->load->database($db['granary'], TRUE);

This means App_Model, and Comp classes all load dbs. But those classes are never used on the ground. They are there for structure as basically abstract or skeleton-like classes.

Also, there is the whole repeating of those three lines for each App_Model (so repeated for every module). Should bump up the three lines to the Parent_App_Model class, but make it conditional so only Task children classes go through those lines. Parent_App_Model should NOT go through the lines itself.

Reported by S. Shehzed · March 8th, 2011 @ 07:20 PM

State: new
Milestone: v0.20
Assigned to: nobody

Activity

No comments yet.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.