← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/config.py

Fix config calls for the actual implementation, including typos in parameters and TransportConfig support.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/config.py
857
857
        for hook in ConfigHooks['remove']:
858
858
            # We're lying here pretending to be a Stack until we get deprecated
859
859
            # -- vila 20110610
860
 
            hook(self, name)
 
860
            hook(self, option_name)
861
861
 
862
862
    def _write_config_file(self):
863
863
        if self.file_name is None:
869
869
        atomic_file.commit()
870
870
        atomic_file.close()
871
871
        osutils.copy_ownership_from_path(self.file_name)
 
872
        for hook in ConfigHooks['save']:
 
873
            # We're lying here pretending to be a Store until we get deprecated
 
874
            # -- vila 20110610
 
875
            hook(self)
872
876
 
873
877
 
874
878
class LockableConfig(IniBasedConfig):
1005
1009
        for hook in ConfigHooks['set']:
1006
1010
            # We're lying here pretending to be a Store until we get deprecated
1007
1011
            # -- vila 20110610
1008
 
            hook(self, name, value)
 
1012
            hook(self, option, value)
1009
1013
 
1010
1014
    def _get_sections(self, name=None):
1011
1015
        """See IniBasedConfig._get_sections()."""
1210
1214
        for hook in ConfigHooks['set']:
1211
1215
            # We're lying here pretending to be a Store until we get deprecated
1212
1216
            # -- vila 20110610
1213
 
            hook(self, name, value)
 
1217
            hook(self, option, value)
1214
1218
 
1215
1219
 
1216
1220
class BranchConfig(Config):
2113
2117
                section_obj = configobj[section]
2114
2118
            except KeyError:
2115
2119
                return default
2116
 
        return section_obj.get(name, default)
 
2120
        value = section_obj.get(name, default)
 
2121
        for hook in ConfigHooks['get']:
 
2122
            # We're lying here pretending to be a Stack until we get deprecated
 
2123
            # -- vila 20110613
 
2124
            hook(self, name, value)        
 
2125
        return value
2117
2126
 
2118
2127
    def set_option(self, value, name, section=None):
2119
2128
        """Set the value associated with a named option.
2127
2136
            configobj[name] = value
2128
2137
        else:
2129
2138
            configobj.setdefault(section, {})[name] = value
 
2139
        for hook in ConfigHooks['set']:
 
2140
            # We're lying here pretending to be a Store until we get deprecated
 
2141
            # -- vila 20110610
 
2142
            hook(self, name, value)
2130
2143
        self._set_configobj(configobj)
2131
2144
 
2132
2145
    def remove_option(self, option_name, section_name=None):
2135
2148
            del configobj[option_name]
2136
2149
        else:
2137
2150
            del configobj[section_name][option_name]
 
2151
        for hook in ConfigHooks['remove']:
 
2152
            # We're lying here pretending to be a Store until we get deprecated
 
2153
            # -- vila 20110613
 
2154
            hook(self, option_name, value)
2138
2155
        self._set_configobj(configobj)
2139
2156
 
2140
2157
    def _get_config_file(self):
2141
2158
        try:
2142
 
            return StringIO(self._transport.get_bytes(self._filename))
 
2159
            f = StringIO(self._transport.get_bytes(self._filename))
 
2160
            for hook in ConfigHooks['load']:
 
2161
                # We're lying here pretending to be a Store until we get
 
2162
                # deprecated -- vila 20110613
 
2163
                hook(self)
 
2164
            return f
2143
2165
        except errors.NoSuchFile:
2144
2166
            return StringIO()
2145
2167
 
2155
2177
        configobj.write(out_file)
2156
2178
        out_file.seek(0)
2157
2179
        self._transport.put_file(self._filename, out_file)
 
2180
        for hook in ConfigHooks['save']:
 
2181
            # We're lying here pretending to be a Store until we get deprecated
 
2182
            # -- vila 20110613
 
2183
            hook(self)
2158
2184
 
2159
2185
 
2160
2186
class Option(object):