Was looking at this again. Fei Long, in your patch, the new code in _do_add_locations() raises exception.Forbidden. The current code raises webob.exc.HTTPForbidden (proceeding if block, when show_multiple_locations is False). Should your patch be using the webob exception in this spot?
Was looking at this again. Fei Long, in your patch, the new code in _do_add_locations() raises exception. Forbidden. The current code raises webob.exc. HTTPForbidden (proceeding if block, when show_multiple_ locations is False). Should your patch be using the webob exception in this spot?