Ticket #173 (new enhancement)
Opened 22 months ago
Remove public properties in favor of accessor methods
| Reported by: | jeff@… | Owned by: | pmjones |
|---|---|---|---|
| Priority: | major | Component: | code |
| Keywords: | Cc: |
Description
Here is a hit list of public properties in Solar. Changing these to accessor methods will be more flexible and future proof. This list excludes controller, model, and cli utility classes. I have not analyzed the impact of any single property.
Solar_Access_Adapter->list
Solar_Auth_Adapter->allow
Solar_Cache_Adapter_Memcache->memcache
Solar_Form->attribs
Solar_Form->elements
Solar_Form->feedback
Solar_Getopt->options
Solar_Http_Request_Adapter->content
Solar_Http_Response->content
Solar_Locale->trans
Solar_Request->env
Solar_Request->get
Solar_Request->post
Solar_Request->cookie
Solar_Request->server
Solar_Request->files
Solar_Request->http
Solar_Request->argv
Solar_Uri->scheme
Solar_Uri->host
Solar_Uri->port
Solar_Uri->user
Solar_Uri->pass
Solar_Uri->path
Solar_Uri->format
Solar_Uri->fragment
Solar_User->auth
Solar_User->role
Solar_User->access
Solar_View_Helper_GetTextRaw->_class
thoughts?
