EJB plugin question

[Mark Struberg]

Hi!

EJBInstanceProxy contains the following method:

    public DeploymentInfo getDeploymentInfo()
    {
        if (deploymentInfo != null)
        {
            return deploymentInfo;
        }
        else if (deploymentId != null)
        {
            ContainerSystem containerSystem = 
SystemInstance.get().getComponent(ContainerSystem.class);
            DeploymentInfo deploymentInfo = 
containerSystem.getDeploymentInfo(deploymentId);
            return deploymentInfo;
        }
        return null;
    }

but deploymentInfo is already a local private member. Shouldn't this set this 
member instead of always asking the container instead?

I for now will rename the return value into deploymentInfoFromContainer to make 
it more clear and to remove the name overlapping...


LieGrue,
strub