#7979 Phone validation interfering with project import

v1.3.2
closed
General
Heith Seewald
2015-09-04
2015-08-31
No
15:18:01,597 ERROR [forgeimporters.base] error registering project: foobar
Traceback (most recent call last):
  File "/var/local/allura/ForgeImporters/forgeimporters/base.py", line 292, in process
    project_name=kw['project_name'])
  File "/var/local/allura/Allura/allura/model/neighborhood.py", line 129, in register_project
    self, shortname, project_name, user or getattr(c, 'user', None), user_project, private_project, apps)
  File "/var/local/forge-classic/ForgeSF/forgesf/sf_providers.py", line 426, in register_project
    self.validate_project(neighborhood, shortname, project_name, user, user_project, private_project)
  File "/var/local/allura/Allura/allura/lib/plugin.py", line 816, in validate_project
    raise forge_exc.ProjectPhoneVerificationError()
ProjectPhoneVerificationError

Discussion

  • Dave Brondsema

    Dave Brondsema - 2015-08-31

    Fix on db/7979 allowing project imports without phone validation. To test, enable phone validation in your .ini config and log in with a user that is not a neighborhood admin and does not have any projects yet. Try to do a project import (of any type). It should fail on master and work on this branch.

     
  • Dave Brondsema

    Dave Brondsema - 2015-08-31
    • status: in-progress --> review
     
  • Heith Seewald

    Heith Seewald - 2015-08-31
    • Reviewer: Heith Seewald
     
  • Heith Seewald

    Heith Seewald - 2015-08-31
    • status: review --> closed
     
  • Heith Seewald

    Heith Seewald - 2015-08-31

    Looks good.

    Merged.

     
  • Dave Brondsema

    Dave Brondsema - 2015-09-04
    • labels: phone, import, sf-current, sf-1 --> phone, import, sf-1
     
  • Dave Brondsema

    Dave Brondsema - 2015-12-08
    • Milestone: unreleased --> v1.3.2
     

Log in to post a comment.