Skip to content
This repository has been archived by the owner on Aug 29, 2023. It is now read-only.

Report Law of Demeter violation #51

Open
edsonmedina opened this issue Jan 7, 2015 · 0 comments
Open

Report Law of Demeter violation #51

edsonmedina opened this issue Jan 7, 2015 · 0 comments
Assignees
Milestone

Comments

@edsonmedina
Copy link
Owner

These:

 $externalObj->subObject->method();
 $a = $externalObj->anotherObj->someproperty;

Violate the Law of Demeter.

But this:

$query->select('*')->from('table')->where('1=1');

Does not (methods instead of properties)

Also this:

$this->internalObj->getSomething();

Because it is an internal object.

@edsonmedina edsonmedina self-assigned this Jan 7, 2015
@edsonmedina edsonmedina added this to the 1.0 milestone Jan 7, 2015
@edsonmedina edsonmedina modified the milestones: 0.4, 0.5 Jan 15, 2015
@edsonmedina edsonmedina changed the title Report external object property access Report Law of Demeter violation Jan 30, 2015
@edsonmedina edsonmedina modified the milestones: 0.5, 0.6 Feb 12, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant