SolarArbiter / solarforecastarbiter-api

HTTP API and database schema for the Solar Forecast Arbiter
https://api.solarforecastarbiter.org
MIT License
10 stars 6 forks source link

UnboundLocalError: local variable 'created_uuid' referenced before assignment #167

Closed sentry-io[bot] closed 4 years ago

sentry-io[bot] commented 4 years ago

I repeated using testing@solarforecastarbiter.org instead of my own account and it worked fine.

Sentry Issue: DASHBOARD-Z

UnboundLocalError: local variable 'created_uuid' referenced before assignment
(4 additional frame(s) were not displayed)
...
  File "flask/app.py", line 1813, in full_dispatch_request
    rv = self.dispatch_request()
  File "flask/app.py", line 1799, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "flask/views.py", line 88, in view
    return self.dispatch_request(*args, **kwargs)
  File "flask/views.py", line 158, in dispatch_request
    return meth(*args, **kwargs)
  File "sfa_dash/blueprints/form.py", line 255, in post
    uuid=created_uuid))
lboeman commented 4 years ago

@wholmgren I made a mistake in the logic in the creation error handling that causes the errors to be passed to the template that assumes successful creation. I'll fix that and push the changes. The db has been nuked since I created the University of Arizona organization. Do you want me to recreate that or should I wait until the dust has settled on the aggregates stuff?

wholmgren commented 4 years ago

Depends on timeframe for aggregates. I need to be able to take screenshots for permissions stuff. Ideally later today, definitely tomorrow.

On Thu, Oct 3, 2019 at 9:45 AM Leland Boeman notifications@github.com wrote:

@wholmgren https://github.com/wholmgren I made a mistake in the logic in the creation error handling that causes the errors to be passed to the template that assumes successful creation. I'll fix that and push the changes. The db has been nuked since I created the University of Arizona organization. Do you want me to recreate that or should I wait until the dust has settled on the aggregates stuff?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/SolarArbiter/solarforecastarbiter-api/issues/167?email_source=notifications&email_token=ABBOER42KKPT7KKZJDIMRFTQMYOQ5A5CNFSM4I45ZN6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAI2K2A#issuecomment-538027368, or mute the thread https://github.com/notifications/unsubscribe-auth/ABBOER7REFHIQCZEJZCKDS3QMYOQ5ANCNFSM4I45ZN6A .

lboeman commented 4 years ago

@wholmgren This error will be caught now. You should see something like this when trying to create a metadata object with an unprivileged account: sitecreationerror

wholmgren commented 4 years ago

just to clarify... I lost the privilege when you erased the database and my account went back to unaffiliated?

On Thu, Oct 3, 2019 at 10:26 AM Leland Boeman notifications@github.com wrote:

@wholmgren https://github.com/wholmgren This error will be caught now. You should see something like this when trying to create a metadata object with an unprivileged account: [image: sitecreationerror] https://user-images.githubusercontent.com/21206164/66149595-47a92c80-e5c8-11e9-8473-ff743f42af2e.png

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/SolarArbiter/solarforecastarbiter-api/issues/167?email_source=notifications&email_token=ABBOERY7DMEFWCYSK42FRY3QMYTNJA5CNFSM4I45ZN6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAI6MEA#issuecomment-538043920, or mute the thread https://github.com/notifications/unsubscribe-auth/ABBOER7THZB3XIOR7PRFYZ3QMYTNJANCNFSM4I45ZN6A .

lboeman commented 4 years ago

Yes.