Skip to content

Entity (flat and attribute)#15

Merged
Dunkon merged 10 commits intomasterfrom
feature/entity_type
Apr 19, 2017
Merged

Entity (flat and attribute)#15
Dunkon merged 10 commits intomasterfrom
feature/entity_type

Conversation

@queyenth
Copy link
Contributor

@queyenth queyenth commented Apr 17, 2017

This feature closes #9 issue.

Queyenth added 8 commits April 12, 2017 16:10
Which implements ArrayAccess interface, and also removed not necessary anymore translateToAttributeName function from AttributeHelper,
Cause it do nothing with AttributeHelper. And extended ProductAbstract from that Entity.

That needs to create new classes and have option to just get column without writing any special SQL. So it’s simple ORM, I guess.
So, in magento, entities can be either attributes based, or flat.
Reason - we should not remove it like this, take a look on deleteCategories test.
@queyenth queyenth added this to the v1.0.0 milestone Apr 17, 2017
@queyenth queyenth requested a review from Dunkon April 17, 2017 10:29
@coveralls
Copy link

coveralls commented Apr 17, 2017

Coverage Status

Coverage decreased (-3.2%) to 93.382% when pulling 9df1bcd on feature/entity_type into 14f27e9 on master.

@coveralls
Copy link

coveralls commented Apr 17, 2017

Coverage Status

Coverage decreased (-1.5%) to 95.177% when pulling 49d1b79 on feature/entity_type into 14f27e9 on master.

@coveralls
Copy link

coveralls commented Apr 18, 2017

Coverage Status

Coverage decreased (-1.4%) to 95.281% when pulling 9ac9b68 on feature/entity_type into 14f27e9 on master.

@queyenth queyenth changed the title Feature/entity type Closes #9 Apr 18, 2017
@queyenth queyenth changed the title Closes #9 Entity (flat and attribute) Apr 18, 2017
@Dunkon Dunkon merged commit 601745c into master Apr 19, 2017
@Dunkon Dunkon deleted the feature/entity_type branch April 19, 2017 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

entity_type_id can be not 4.

3 participants