[Trac-tickets] [The Trac Project] #3037: Multiple bugs in trac.config

[The Trac Project]

#3037: Multiple bugs in trac.config
---------------------+------------------------------------------------------
 Reporter:  athomas  |       Owner:  jonas        
     Type:  defect   |      Status:  new          
 Priority:  high     |   Milestone:  0.10         
Component:  general  |     Version:  devel        
 Severity:  major    |    Keywords:  global config
---------------------+------------------------------------------------------
 While testing some global configuration changes, I noticed they weren't
 working with InterTrac. After some investigation it seems there are a few
 problems with the current config module:

  1. `Section.__contains__()` does not use the global config, nor does
 `Section.get()`
  1. `ConfigParser` ignores case, but `config.Configuration._defaults` does
 not.
  1. `ConfigParser.read()` overlays previous configuration. The issue with
 this is that we use `read()` when a configuration modification is
 detected. Deleted options are not removed.

 The attached patch fixes the first two issues. It also updates one of the
 config unit tests, as it can now fail if a global configuration file has
 entries (NB. I'm not sure whether the ''fix'' in the unit test is too
 hackish?)

 The third issue would require some kind of dirty configuration list, for
 both the site and env configs. I'm not sure the issue is bad enough to
 warrant the ugliness.

-- 
Ticket URL: <http://projects.edgewall.com/trac/ticket/3037>
The Trac Project <http://trac.edgewall.com/>