We should include the default mount point in the pre-submit error checking.  If 
you leave it blank, the task can fail later with:

~~~~
Traceback (most recent call last):
  File 
"/home/dbrondsema/dbrondsema-1019/forge/Allura/allura/model/monq_model.py", 
line 258, in __call__
    self.result = func(*self.args, **self.kwargs)
  File 
"/nfs/home/dbrondsema/googlecodewikiimporter/googlecodewikiimporter/importer.py",
 line 47, in import_tool
    GoogleCodeWikiImporter().import_tool(c.project, c.user, **kw)
  File 
"/nfs/home/dbrondsema/googlecodewikiimporter/googlecodewikiimporter/importer.py",
 line 203, in import_tool
    mount_label=mount_label or 'Wiki',
  File "/home/dbrondsema/dbrondsema-1019/forge/Allura/allura/model/project.py", 
line 606, in install_app
    raise exceptions.ToolError(str(e))
ToolError: Mount point "wiki" is already in use
~~~~


---

** [tickets:#6526] Fixes for individual tool importers**

**Status:** in-progress
**Labels:** import 
**Created:** Wed Aug 07, 2013 03:43 PM UTC by Dave Brondsema
**Last Updated:** Tue Aug 27, 2013 04:57 PM UTC
**Owner:** Tim Van Steenburgh

Individual tool importer controllers will need their `import_tool` calls 
changed to be tasks.  <s>Perhaps create the tool right away, kick off the task, 
then redirect to the new tool with a flash message.</s>

Other fixes are likely needed too, once this controller is exposed and can be 
tested.


---

Sent from sourceforge.net because allura-dev@incubator.apache.org is subscribed 
to https://sourceforge.net/p/allura/tickets/

To unsubscribe from further messages, a project admin can change settings at 
https://sourceforge.net/p/allura/admin/tickets/options.  Or, if this is a 
mailing list, you can unsubscribe from the mailing list.

Reply via email to