c2c-oerpscenario team mailing list archive

["Olivier Dony \(OpenERP\)" <716710@xxxxxxxxxxxxxxxxxx>]

@Matteo: your suggested patch is good too, and it may help detect
incorrect configuration parameters. However we would like to leave the
flexibility to system administrators to use paths that do not
necessarily contain addons yet, and trust them to use the proper
settings that work for them.

-- 
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.
https://bugs.launchpad.net/bugs/716710

Title:
  [TRUNK 3353] dont more support multiple addons path

Status in OpenERP Server:
  Fix Released

Bug description:
  config.py

  if not self.options['addons_path'] or self.options['addons_path']=='None':
              self.options['addons_path'] = os.path.join(self.options['root_path'], 'addons')
          else:
              self.options['addons_path'] = os.path.abspath(os.path.expanduser(os.path.expandvars(self.options['addons_path'])))

  The Else dont support multipath (supporting before, and still support
  after my correction)

  Correction

   if not self.options['addons_path'] or self.options['addons_path']=='None':
              self.options['addons_path'] = os.path.join(self.options['root_path'], 'addons')
          else:
              o = self.options['addons_path']
              self.options['addons_path'] = ''
              for x in o.split(','):
                  self.options['addons_path'] = ','.join([self.options['addons_path'], os.path.abspath(os.path.expanduser(os.path.expandvars(x)))])

  So is not the better way, but atm i dont care i need that work.

  ps : i know this is a trunk, but before commit branch on trunk no test
  done to ensure all functionality follow ?