Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Amendments to class properties #58

Closed
GaryJones opened this Issue · 3 comments

2 participants

@GaryJones
Collaborator

Change each of the class properties from using the var keyword to the private or protected (if we expect the class might be extended upon) keyword, since they shouldn't be able to be accessed from outside the class.

@GaryJones GaryJones was assigned
@thomasgriffin

I access these properties within the new classes that aid bulk installation/activation. Not directly, mind you, but I do access them via the $instance property. Will setting these to protected or private affect the current usage?

@thomasgriffin

Setting this to 3.0.0 as it is not urgent for the next version release (2.2.0).

@GaryJones
Collaborator

Possibly - but lets set them to public for now, and consider whether we want them as private in 3.0.0

Changing this milestone back to 2.2.0 for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.