[GitHub] stratos pull request: Fixed deactivating non-exist tenant in cli

2015-05-18 Thread pubudu538
Github user pubudu538 closed the pull request at: https://github.com/apache/stratos/pull/329 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] stratos pull request: Fixed deactivating non-exist tenant in cli

2015-05-18 Thread imesh
Github user imesh commented on the pull request: https://github.com/apache/stratos/pull/329#issuecomment-103052866 Thanks Pubudu! I have now merged this to master branch, please close the pull request manually. Thanks --- If your project is set up for it, you can reply to this em

[GitHub] stratos pull request: Fixed deactivating non-exist tenant in cli

2015-05-18 Thread pubudu538
Github user pubudu538 commented on the pull request: https://github.com/apache/stratos/pull/329#issuecomment-103046924 I have fixed both tenant activate and deactivate cli issues. Please review and merge. --- If your project is set up for it, you can reply to this email and have your

Re: [GitHub] stratos pull request: Fixed deactivating non-exist tenant in cli

2015-05-18 Thread Pubudu Gunatilaka
This exception is needed for the non exist tenant domain. As we discussed offline, i will change this particular exception to InvalidDomainException and send the pull request again. On Mon, May 18, 2015 at 2:36 PM, lahirus wrote: > Github user lahirus commented on a diff in the pull request: > >

[GitHub] stratos pull request: Fixed deactivating non-exist tenant in cli

2015-05-18 Thread lahirus
Github user lahirus commented on a diff in the pull request: https://github.com/apache/stratos/pull/329#discussion_r30491310 --- Diff: components/org.apache.stratos.rest.endpoint/src/main/java/org/apache/stratos/rest/endpoint/api/StratosApiV41Utils.java --- @@ -3170,7 +3170,27 @@

[GitHub] stratos pull request: Fixed deactivating non-exist tenant in cli

2015-05-18 Thread pubudu538
GitHub user pubudu538 opened a pull request: https://github.com/apache/stratos/pull/329 Fixed deactivating non-exist tenant in cli fixed deactivating non-exist tenant in cli. In the current implementation it doesn't check for the existence of the tenant. Tenant id becomes -1 for non