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?

Attachments

Add/Change #173 (Remove public properties in favor of accessor methods)

Author


E-mail address and user name can be saved in the Preferences.


Action
as new
 
Note: See TracTickets for help on using tickets.