LocalContextConnection::injectSessions() - Code Metrics - appserver-io/rmi - Measure and Improve Code Quality continuously with Scrutinizer

LocalContextConnection::injectSessions() A
last analyzed 2018-07-17 19:20 UTC

↳ Parent: LocalContextConnection

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	1

<?php

/**
 * AppserverIo\PersistenceContainerClient\Context\LocalContextConnection
 *
 * NOTICE OF LICENSE
 *
 * This source file is subject to the Open Software License (OSL 3.0)
 * that is available through the world-wide-web at this URL:
 * http://opensource.org/licenses/osl-3.0.php
 *
 * PHP version 5
 *
 * @author    Tim Wagner <[email protected]>
 * @copyright 2015 TechDivision GmbH <[email protected]>
 * @license   http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
 * @link      https://github.com/appserver-io/rmi
 * @link      http://www.appserver.io
 */

namespace AppserverIo\RemoteMethodInvocation;

use AppserverIo\Collections\CollectionInterface;
use AppserverIo\Psr\Application\ApplicationInterface;
use AppserverIo\Psr\EnterpriseBeans\BeanContextInterface;

/**
 * Connection implementation to invoke a local method call.
 *
 * @author    Tim Wagner <[email protected]>
 * @copyright 2015 TechDivision GmbH <[email protected]>
 * @license   http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
 * @link      https://github.com/appserver-io/rmi
 * @link      http://www.appserver.io
 *
 * @deprecated Use \AppserverIo\RemoteMethodInvocation\LocalProxy instead
 */
class LocalContextConnection implements ConnectionInterface
{

    /**
     * The storage for the sessions.
     *
     * @var \AppserverIo\Collections\CollectionInterface
     */
    protected $sessions = null;

    /**
     * The application instance.
     *
     * @var \AppserverIo\Psr\Application\ApplicationInterface
     */
    protected $application;

    /**
     * The bean manager instance to load the local bean instance with.
     *
     * @var \AppserverIo\Psr\EnterpriseBeans\BeanContextInterface
     */
    protected $beanManager;

    /**
     * The local bean instance we're the proxy for.
     *
     * @var object
     */
    protected $instance;

    /**
     * Injects the application instance for the local connection.
     *
     * @param \AppserverIo\Psr\Application\ApplicationInterface $application The application instance
     *
     * @return void
     */
    public function injectApplication(ApplicationInterface $application)
    {
        $this->application = $application;
    }

    /**
     * Returns the application instance.
     *
     * @return \AppserverIo\Psr\Application\ApplicationInterface The application instance
     */
    public function getApplication()
    {
        return $this->application;
    }

    /**
     * Injects the collection for the sessions.
     *
     * @param \AppserverIo\Collections\CollectionInterface $sessions The collection for the sessions
     *
     * @return void
     */
    public function injectSessions(CollectionInterface $sessions)
    {
        $this->sessions = $sessions;
    }

    /**
     * Returns the collection with the sessions.
     *
     * @return \AppserverIo\Collections\CollectionInterface The collection with the sessions
     */
    public function getSessions()
    {
        return $this->sessions;
    }

    /**
     * Sends the remote method call to the container instance.
     *
     * @param \AppserverIo\RemoteMethodInvocation\RemoteMethodInterface $remoteMethod The remote method instance
     *
     * @return mixed The response from the container
     * @see AppserverIo\RemoteMethodInvocation\ConnectionInterface::send()
     */
    public function send(RemoteMethodInterface $remoteMethod)
    {
        return $this->getApplication()->search(BeanContextInterface::IDENTIFIER)->invoke($remoteMethod, $this->getSessions());

    }

    /**
     * Initializes a new session instance.
     *
     * @return \AppserverIo\RemoteMethodInvocation\SessionInterface The session instance
     * @see \AppserverIo\RemoteMethodInvocation\ConnectionInterface::createContextSession()
     */
    public function createContextSession()
    {
        $this->sessions->add($session = new ContextSession($this));
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        return $session;
    }
}


1			<?php
2
3			/**
4			* AppserverIo\PersistenceContainerClient\Context\LocalContextConnection
5			*
6			* NOTICE OF LICENSE
7			*
8			* This source file is subject to the Open Software License (OSL 3.0)
9			* that is available through the world-wide-web at this URL:
10			* http://opensource.org/licenses/osl-3.0.php
11			*
12			* PHP version 5
13			*
14			* @author Tim Wagner <[email protected]>
15			* @copyright 2015 TechDivision GmbH <[email protected]>
16			* @license http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
17			* @link https://github.com/appserver-io/rmi
18			* @link http://www.appserver.io
19			*/
20
21			namespace AppserverIo\RemoteMethodInvocation;
22
23			use AppserverIo\Collections\CollectionInterface;
24			use AppserverIo\Psr\Application\ApplicationInterface;
25			use AppserverIo\Psr\EnterpriseBeans\BeanContextInterface;
26
27			/**
28			* Connection implementation to invoke a local method call.
29			*
30			* @author Tim Wagner <[email protected]>
31			* @copyright 2015 TechDivision GmbH <[email protected]>
32			* @license http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
33			* @link https://github.com/appserver-io/rmi
34			* @link http://www.appserver.io
35			*
36			* @deprecated Use \AppserverIo\RemoteMethodInvocation\LocalProxy instead
37			*/
38			class LocalContextConnection implements ConnectionInterface
39			{
40
41			/**
42			* The storage for the sessions.
43			*
44			* @var \AppserverIo\Collections\CollectionInterface
45			*/
46			protected $sessions = null;
47
48			/**
49			* The application instance.
50			*
51			* @var \AppserverIo\Psr\Application\ApplicationInterface
52			*/
53			protected $application;
54
55			/**
56			* The bean manager instance to load the local bean instance with.
57			*
58			* @var \AppserverIo\Psr\EnterpriseBeans\BeanContextInterface
59			*/
60			protected $beanManager;
61
62			/**
63			* The local bean instance we're the proxy for.
64			*
65			* @var object
66			*/
67			protected $instance;
68
69			/**
70			* Injects the application instance for the local connection.
71			*
72			* @param \AppserverIo\Psr\Application\ApplicationInterface $application The application instance
73			*
74			* @return void
75			*/
76			public function injectApplication(ApplicationInterface $application)
77			{
78			$this->application = $application;
79			}
80
81			/**
82			* Returns the application instance.
83			*
84			* @return \AppserverIo\Psr\Application\ApplicationInterface The application instance
85			*/
86			public function getApplication()
87			{
88			return $this->application;
89			}
90
91			/**
92			* Injects the collection for the sessions.
93			*
94			* @param \AppserverIo\Collections\CollectionInterface $sessions The collection for the sessions
95			*
96			* @return void
97			*/
98			public function injectSessions(CollectionInterface $sessions)
99			{
100			$this->sessions = $sessions;
101			}
102
103			/**
104			* Returns the collection with the sessions.
105			*
106			* @return \AppserverIo\Collections\CollectionInterface The collection with the sessions
107			*/
108			public function getSessions()
109			{
110			return $this->sessions;
111			}
112
113			/**
114			* Sends the remote method call to the container instance.
115			*
116			* @param \AppserverIo\RemoteMethodInvocation\RemoteMethodInterface $remoteMethod The remote method instance
117			*
118			* @return mixed The response from the container
119			* @see AppserverIo\RemoteMethodInvocation\ConnectionInterface::send()
120			*/
121			public function send(RemoteMethodInterface $remoteMethod)
122			{
123			return $this->getApplication()->search(BeanContextInterface::IDENTIFIER)->invoke($remoteMethod, $this->getSessions());
			0 ignored issues – show Bug introduced 2016-04-29 08:30 UTC by Report Bug Copy Issue Report The method `search()` does not seem to exist on `object<AppserverIo\Psr\A...n\ApplicationInterface>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
124			}
125
126			/**
127			* Initializes a new session instance.
128			*
129			* @return \AppserverIo\RemoteMethodInvocation\SessionInterface The session instance
130			* @see \AppserverIo\RemoteMethodInvocation\ConnectionInterface::createContextSession()
131			*/
132			public function createContextSession()
133			{
134			$this->sessions->add($session = new ContextSession($this));
			0 ignored issues – show Bug introduced 2015-12-21 13:46 UTC by Report Bug Copy Issue Report It seems like you code against a concrete implementation and not the interface `AppserverIo\Collections\CollectionInterface` as the method `add()` does only exist in the following implementations of said interface: `AppserverIo\Collections\ArrayList`, `AppserverIo\Collections\Dictionary`, `AppserverIo\Collections\HashMap`, `AppserverIo\Collections\IdentityDictionary`, `AppserverIo\Collections\TreeMap`, `AppserverIo\RemoteMethodInvocation\ContextSession`. Let’s take a look at an example: interface User { /** @return string / public function getPassword(); } class MyUser implements User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different implementation of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the interface: interface User { /** @return string / public function getPassword(); /* @return string */ public function getDisplayName(); } Loading history...
135			return $session;
136			}
137			}
138

appserver-io / rmi

LocalContextConnection::injectSessions() A last analyzed 2018-07-17 19:20 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like

LocalContextConnection::injectSessions() A
last analyzed 2018-07-17 19:20 UTC