zombor / auto-modeler-demo Goto Github PK

https://github.com/zombor/Auto-Modeler-Demo/blob/master/application/classes/context/user/add.php#L104

Should this not read $this->_data = $data ?

I know it's nothing to do with your project, but it might be worth noting the following:

Using your instructions: https://github.com/zombor/Auto-Modeler-Demo/#specs
I run into this issue, but their work around works: composer/composer#690 (comment)

Could you add AutoModeler_ORM to your demo for when you put it on your domain.
It might be useful for those who just started using Kohana.

Models used by repositories must have a defined __construct function, otherwise when query builder casts result as_object, you will get this error:

Class [...] does not have a constructor hence you cannot use ctor_param

What this example is really lacking is some relational examples. Like displaying the groups the user is in. And adding a group for to a user.

Also it would be great to have some examples of what should be in the Model and what in the gateway. Maybe connecting a first name, middle name and last name.

This is just an idea but maybe it's a good way to switch the factory and the actual context around. This is just for naming and order sense. It makes things feel right or better IMO.

So we call the factory: Context_User_Add and call it by new Context_User_Add
And we call the context Context_User_Add_Case and we call it in Context_User_Add

Controller -> Context_User_Add -> Context_User_Add_Case

Pre's
No _Factory in the naming
The case is a directory lower and the requester is 1 directory up. (easier browsing)

https://github.com/zombor/Auto-Modeler-Demo/blob/master/composer.json#L5 "cordoval" seems to be out of date.
The issue is a breaking change as described here:
https://groups.google.com/forum/#!msg/composer-dev/fWIs3KocwoA/nU3aLko9LhQJ

There must be a newer version to point composer to?