concrete classes should include generated classes

Bug #340114 reported by Anthony Bush
2
Affects Status Importance Assigned to Milestone
CoughPHP
New
Low
Anthony Bush

Bug Description

Concrete classes should include generated classes.

* For those not using Autoloader, won't have to include two files every-time they just want to use one.
* For those using Autoloader, lightens the load for it.

A question to ask, however, is whether or not the concrete collection class should include the concrete element class. It probably should, since the moment you try to do anything on it the other class would be required.

So, try this:

// ClassName.class.php
require_once($pathToGenerated . 'ClassName_Generated.class.php');
class ClassName extends ClassName_Generated ...

// ClassName_Collection.class.php
require_once($pathToConcrete . 'ClassName.class.php');
require_once($pathToGenerated . 'ClassName_Collection_Generated.class.php');
class ClassName_Collection extends ClassName_Collection_Generated ...

I don't mean to literally use $pathTo* variables, but instead to have the generator put the actual path, whatever that happens to be according to the config file at time of generation.

Anthony Bush (awbush)
Changed in coughphp:
assignee: nobody → awbush
importance: Undecided → Low
milestone: none → 1.4
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.