CrisisCleanup / crisiscleanup-1

[OLD] Legacy Crisis Cleanup on GAE/Python
https://sandy-disaster-recovery.appspot.com
Other
8 stars 4 forks source link

Organization Info 500 Error #202

Closed aarontitus closed 11 years ago

aarontitus commented 11 years ago

Original author: v...@aarontitus.net (February 25, 2013 17:06:59)

What steps will reproduce the problem?

  1. Login to Hattiesburg, MS Tornado as admin
  2. Navigate to https://www.crisiscleanup.org/see-all-contacts
  3. Click on link to https://www.crisiscleanup.org/organization-info?organization=MS%20Conf.%20UMC%20Disaster%20Response%20an%20UMCOR%20Partner

What is the expected output? What do you see instead? Get a 500 error. Is it because of the special character (".") in the URL?

Please use labels and text to provide additional information.

Original issue: http://code.google.com/p/sandy-disaster-recovery/issues/detail?id=202

aarontitus commented 11 years ago

From cpw...@gmail.com on February 26, 2013 11:35:09 Fixed as of r188, by looking up organization by id.

(This may need to be done elsewhere, or differently - to be discussed.)

aarontitus commented 11 years ago

From v...@aarontitus.net on February 26, 2013 13:52:40 I assume r188 is not on the sandbox yet, correct? I get the same error when navigating to: https://sandy-helping-hands.appspot.com/organization-info?organization=Test21Feb

aarontitus commented 11 years ago

From cpw...@gmail.com on February 26, 2013 14:28:31 What page is that link from?

That link should no longer be generated - it now uses IDs rather than names.

aarontitus commented 11 years ago

From cpw...@gmail.com on February 26, 2013 14:32:36 I caught it - it's on https://sandy-helping-hands.appspot.com/see-all-contacts rather than https://sandy-helping-hands.appspot.com/organization-info - my mistake.

aarontitus commented 11 years ago

From cpw...@gmail.com on February 26, 2013 14:47:59 Fixed on both apps, e.g. https://www.crisiscleanup.org/see-all-contacts

aarontitus commented 11 years ago

From cpw...@gmail.com on February 26, 2013 14:48:18 r189, version 98

aarontitus commented 11 years ago

From cpw...@gmail.com on March 05, 2013 09:55:49 Complete, closing.