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.