Comment 3 for bug 1042836

There is a similar problem when you export and import content via portal_setup. The export tarball will have a line with:

allowDiscussion: False

(Or True of course.) When you import this tarball you get a traceback:

2013-01-17 18:42:38 ERROR Zope.SiteErrorLog 1358444558.710.826857902685 http://127.0.0.1:8080/Plonelowerprop/portal_setup/manage_importTarball
Traceback (innermost last):
  Module ZPublisher.Publish, line 126, in publish
  Module ZPublisher.mapply, line 77, in mapply
  Module ZPublisher.Publish, line 46, in call_object
  Module Products.GenericSetup.tool, line 565, in manage_importTarball
  Module Products.GenericSetup.tool, line 350, in runAllImportStepsFromProfile
  Module Products.GenericSetup.tool, line 1100, in _runImportStepsFromContext
  Module Products.GenericSetup.tool, line 1015, in _doRunImportStep
   - __traceback_info__: content
  Module Products.CMFCore.exportimport.content, line 34, in importSiteStructure
  Module Products.CMFCore.exportimport.content, line 190, in import_
  Module Products.GenericSetup.content, line 408, in import_
  Module Products.Archetypes.WebDAVSupport, line 121, in PUT
  Module Products.Archetypes.utils, line 123, in mapply
  Module Products.Marshall.marshaller, line 107, in demarshall
  Module Products.Marshall.marshaller, line 90, in delegate
   - __traceback_info__: (<Products.Archetypes.Marshall.RFC822Marshaller instance at 0x10467e170>, 'demarshall', <ATDocument at /Plonelowerprop/page>, {'mimetype': None, 'REQUEST': <Products.GenericSetup.content.FauxDAVRequest instance at 0x1069396c8>, 'RESPONSE': <Products.GenericSetup.content.FauxDAVResponse instance at 0x1069390e0>, 'context': <ATDocument at /Plonelowerprop/page>, 'filename': 'page'})
  Module Products.Archetypes.utils, line 123, in mapply
  Module Products.Archetypes.Marshall, line 236, in demarshall
  Module Products.Archetypes.ExtensibleMetadata, line 287, in allowDiscussion
  Module Products.CMFDefault.DiscussionTool, line 82, in overrideDiscussionFor
ValueError: invalid literal for int() with base 10: 'False'

Maybe it needs to be fixed somewhere else, but it works as expected for other boolean fields as far as I see.
So it seems easiest to add a fix in CMFDefault. I will do that on branch 2.2 and trunk.