Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
From: borisroman <git@git.apache.org>
To: dev@cloudstack.apache.org
Reply-To: dev@cloudstack.apache.org
Message-ID: <git-pr-878-cloudstack@git.apache.org>
Subject: [GitHub] cloudstack pull request: [4.6][BLOCKER]CLOUDSTACK-8890:
 Added isEm...
Content-Type: text/plain
Date: Wed, 23 Sep 2015 13:17:10 +0000 (UTC)

GitHub user borisroman opened a pull request:

    https://github.com/apache/cloudstack/pull/878

    [4.6][BLOCKER]CLOUDSTACK-8890: Added isEmpty() check to prevent nullPointerException.

    Check if the list is empty before trying to get the first entry. If the list is empty, in example when dealing with projects, it will user the caller user id.
    
    Tests to verify working order:
    1. Deploy ACS
    2. Create project
    3. Create resource in project -> Should succeed!

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/borisroman/cloudstack CLOUDSTACK-8890

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/878.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #878
    
----
commit 80cb3adad079be639331046f844b34a69079c224
Author: Boris Schrijver <boris@pcextreme.nl>
Date:   2015-09-23T13:01:03Z

    Added isEmpty() check to prevent nullPointerException.

----


---
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 enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---