diff options
author | Kevin Smith <git@kismith.co.uk> | 2010-09-15 16:46:52 (GMT) |
---|---|---|
committer | Kevin Smith <git@kismith.co.uk> | 2010-09-15 16:47:53 (GMT) |
commit | 8e596c2c0c9155830bb6e00204301bb0333274a5 (patch) | |
tree | b1a01d330ef9fb45c623aaba503ad0e9797c3e91 /Swift/Controllers | |
parent | 5159571a0464c28ce376de22faaa52066168a205 (diff) | |
download | swift-8e596c2c0c9155830bb6e00204301bb0333274a5.zip swift-8e596c2c0c9155830bb6e00204301bb0333274a5.tar.bz2 |
Display useful errors in chats when server doesn't include <text>.
Resolves: #264
Diffstat (limited to 'Swift/Controllers')
-rw-r--r-- | Swift/Controllers/Chat/ChatControllerBase.cpp | 34 |
1 files changed, 17 insertions, 17 deletions
diff --git a/Swift/Controllers/Chat/ChatControllerBase.cpp b/Swift/Controllers/Chat/ChatControllerBase.cpp index 249c433..ffaf3af 100644 --- a/Swift/Controllers/Chat/ChatControllerBase.cpp +++ b/Swift/Controllers/Chat/ChatControllerBase.cpp @@ -190,28 +190,28 @@ String ChatControllerBase::getErrorMessage(boost::shared_ptr<ErrorPayload> error } else { switch (error->getCondition()) { - case ErrorPayload::BadRequest: return defaultMessage; break; - case ErrorPayload::Conflict: return defaultMessage; break; - case ErrorPayload::FeatureNotImplemented: return defaultMessage; break; - case ErrorPayload::Forbidden: return defaultMessage; break; + case ErrorPayload::BadRequest: return "Bad request"; break; + case ErrorPayload::Conflict: return "Conflict"; break; + case ErrorPayload::FeatureNotImplemented: return "This feature is not implemented"; break; + case ErrorPayload::Forbidden: return "Forbidden"; break; case ErrorPayload::Gone: return "Recipient can no longer be contacted"; break; case ErrorPayload::InternalServerError: return "Internal server error"; break; - case ErrorPayload::ItemNotFound: return defaultMessage; break; - case ErrorPayload::JIDMalformed: return defaultMessage; break; + case ErrorPayload::ItemNotFound: return "Item not found"; break; + case ErrorPayload::JIDMalformed: return "JID Malformed"; break; case ErrorPayload::NotAcceptable: return "Message was rejected"; break; - case ErrorPayload::NotAllowed: return defaultMessage; break; - case ErrorPayload::NotAuthorized: return defaultMessage; break; - case ErrorPayload::PaymentRequired: return defaultMessage; break; + case ErrorPayload::NotAllowed: return "Not allowed"; break; + case ErrorPayload::NotAuthorized: return "Not authorized"; break; + case ErrorPayload::PaymentRequired: return "Payment is required"; break; case ErrorPayload::RecipientUnavailable: return "Recipient is unavailable."; break; - case ErrorPayload::Redirect: return defaultMessage; break; - case ErrorPayload::RegistrationRequired: return defaultMessage; break; + case ErrorPayload::Redirect: return "Redirect"; break; + case ErrorPayload::RegistrationRequired: return "Registration required"; break; case ErrorPayload::RemoteServerNotFound: return "Recipient's server not found."; break; - case ErrorPayload::RemoteServerTimeout: return defaultMessage; break; - case ErrorPayload::ResourceConstraint: return defaultMessage; break; - case ErrorPayload::ServiceUnavailable: return defaultMessage; break; - case ErrorPayload::SubscriptionRequired: return defaultMessage; break; - case ErrorPayload::UndefinedCondition: return defaultMessage; break; - case ErrorPayload::UnexpectedRequest: return defaultMessage; break; + case ErrorPayload::RemoteServerTimeout: return "Remote server timeout"; break; + case ErrorPayload::ResourceConstraint: return "The server is low on resources"; break; + case ErrorPayload::ServiceUnavailable: return "The service is unavailable"; break; + case ErrorPayload::SubscriptionRequired: return "A subscription is required"; break; + case ErrorPayload::UndefinedCondition: return "Undefined condition"; break; + case ErrorPayload::UnexpectedRequest: return "Unexpected request"; break; } } return defaultMessage; |