Single table inheritance.

Question

Single table inheritance.

SamMousa opened this issue 8 years ago · comments

Regarding: https://github.com/samdark/yii2-cookbook/blob/master/book/ar-single-table-inheritance.md

I have a package that simplifies STI and adds some more features that you don't get when using the method described codebook. It is located here: https://github.com/SAM-IT/yii2-magic

Some benefits:

Uses configuration in a single place (the base class).
Supports search models (ie classes that extend from subclasses of the base class).
Implemented using traits so easy to combine with custom ActiveRecord and / or ActiveQuery implementations.

Alexander Makarov · Answer 1 · Thu May 12 2016 22:41:47 GMT+0800 (China Standard Time)

I'd split that repo into separate packages in the first place...

Alexander Makarov · Answer 2 · Thu May 12 2016 22:43:20 GMT+0800 (China Standard Time)

I still think that recipe is valid because it shows how to hack on AR.

Sam · Answer 3 · Thu May 12 2016 22:56:41 GMT+0800 (China Standard Time)

I don't doubt the validity of the recipe ^^.

For now I prefer to maintain it as a single collection; if at some point more people start using it and bugs are getting reported / PRs made I'll refactor it to use a meta-package and separate packages.

Just wanted to mention it here so people using the cookbook could find the package.

Alexander Makarov · Answer 4 · Thu May 12 2016 23:00:27 GMT+0800 (China Standard Time)

So any things in the recipe itself which are wrong? If yes, would be great to fix these...

Sam · Answer 5 · Thu May 12 2016 23:50:37 GMT+0800 (China Standard Time)

It requires changes in every subclass. So it's not wrong, but not (imo) the cleanest.
Feel free to close this if you're happy with how it is though :)

Alexander Makarov · Answer 6 · Fri May 13 2016 00:00:08 GMT+0800 (China Standard Time)

I think it's OK. Properties could be extracted from it if there are many such classes which usually isn't the case.