fix(class): property visibility checks #375

[kakserpom]

Fixes #375

[coveralls]

Pull Request Test Coverage Report for Build 20547658374

Details

• 1 of 56 (1.79%) changed or added relevant lines in 2 files are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage decreased (-0.4%) to 35.968%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/types/array/mod.rs	1	2	50.0%
src/zend/handlers.rs	0	54	0.0%

Files with Coverage Reduction	New Missed Lines	%
src/zend/handlers.rs	1	0.0%

Totals
Change from base Build 20542278985:	-0.4%
Covered Lines:	1811
Relevant Lines:	5035

---

💛 - Coveralls

[Xenira]

LGTM. Could you remove the unrelated docblock changes from this PR?

[Xenira]

rustfmt did that. Is that a problem?

Just to make history cleaner and prevent conflicts. Not a big thing, and usually fine. Just saw a lot of those changes in your PRs and in that case its better to do one PR containing all rustfmt changes.