Issue Details (XML | Word | Printable)

Key: MODPYTHON-127
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Graham Dumpleton
Reporter: Graham Dumpleton
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
mod_python

Use namespace for mod_python PythonOption settings.

Created: 10/Feb/06 07:30 AM   Updated: 05/Apr/07 11:48 AM
Return to search
Component/s: core
Affects Version/s: 3.3.x
Fix Version/s: 3.3.1

Time Tracking:
Not Specified

Resolution Date: 27/Oct/06 08:45 AM


 Description  « Hide
In the interests of avoiding name clashes, I want to push that where mod_python uses its own PythonOption settings, that they use a namespace. For example:

    PythonOption mod_python.session_cookie_name ...
    PythonOption mod_python.ApplicationPath ...
    PythonOption mod_python.session_dbm ...
    PythonOption mod_python.session_fast_cleanup ...
    etc ....

If appropriate for mod_python, multiple levels of naming should be used. For example, "session_fast_cleanup" is actually related to FileSession, so perhaps it should be:

  PythonOption mod_python.Session.cookie_name ...
  PythonOption mod_python.Session.application_path ...
  PythonOption mod_python.DbmSession.database ...
  PythonOption mod_python.FileSession.fast_cleanup ...

Thus, class name is interjected as second level in name. Also would like to see final attribute name settle on lower case with underscore between distinct words.

We can support old names in mod_python for the time being but should deprecate them.

Any third party package developers should be strongly encouraged to also put any of their own PythonOption settings names in their own unique namespace.

Mailing list thread where this was first proposed, and in case there were followups of interest, was:

  http://www.modpython.org/pipermail/mod_python/2006-February/020213.html



 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
No work has yet been logged on this issue.