plone / plone.app.ldap

Plone UI and integration of the functionality provided by LDAPMultiPlugins, LDAPUserFolder and PloneLDAP
3 stars 10 forks source link

Control panel configlet not removed after uninstall #26

Open hvelarde opened 9 years ago

hvelarde commented 9 years ago

See: https://travis-ci.org/plone/plone.app.ldap/builds/70302789

adaugherity commented 9 years ago

Duplicate of #14. Not sure which to prefer since that one has the traceback and this one has the failed test skip.

laulaz commented 4 years ago

For the record, this only works in Plone > 5.1 (https://github.com/plone/Products.CMFPlone/commit/d011a517dfe5c31c57100d3922fcc00a171fc7d8).

Control panel configlet is not removed in previous Plone versions, leading to a traceback :

Traceback (innermost last):
  Module ZPublisher.Publish, line 138, in publish
  Module ZPublisher.mapply, line 77, in mapply
  Module ZPublisher.Publish, line 48, in call_object
  Module Products.GenericSetup.tool, line 570, in manage_importAllSteps
  Module Products.GenericSetup.tool, line 379, in runAllImportStepsFromProfile
   - __traceback_info__: profile-plone.app.ldap:uninstall
  Module Products.GenericSetup.tool, line 1414, in _runImportStepsFromContext
  Module Products.GenericSetup.tool, line 1226, in _doRunImportStep
   - __traceback_info__: controlpanel
  Module Products.CMFPlone.exportimport.controlpanel, line 162, in importControlPanel
  Module Products.GenericSetup.utils, line 851, in importObjects
   - __traceback_info__: portal_controlpanel
  Module Products.GenericSetup.utils, line 512, in _importBody
  Module Products.CMFPlone.exportimport.controlpanel, line 55, in _importNode
  Module Products.CMFPlone.exportimport.controlpanel, line 64, in _initProvider
  Module Products.CMFPlone.exportimport.controlpanel, line 151, in _initConfiglets
  Module Products.CMFPlone.PloneControlPanel, line 235, in addAction
ValueError: A name is required.