← Back to team overview

c2c-oerpscenario team mailing list archive

[Bug 716710] Re: [TRUNK 3353] dont more support multiple addons path

 

Hello,

The bug is not fixed the function _check_addons_path is missing a loop on the split of ',' :
So when using multiple addons path the test:   
     if not os.path.exists(res):
            raise optparse.OptionValueError("option %s: no such directory: %r" % (opt, value))
fails

My two cent

Nicolas

-- 
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 ?





References