Completed
Push — master ( 14b43f...b2cd1e )
by Damien
09:39
created
src/services/Metadata.php 1 patch
Unused Use Statements   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -2,10 +2,6 @@  discard block
 block discarded – undo
2 2
 
3 3
 namespace flipbox\saml\core\services;
4 4
 
5
-use craft\base\Component;
6
-use flipbox\keychain\records\KeyChainRecord;
7
-use flipbox\saml\core\helpers\SecurityHelper;
8
-use flipbox\saml\core\models\SettingsInterface;
9 5
 use SAML2\Certificate\Key;
10 6
 use SAML2\Constants;
11 7
 use SAML2\XML\ds\KeyInfo;
@@ -17,6 +13,10 @@  discard block
 block discarded – undo
17 13
 use SAML2\XML\md\KeyDescriptor;
18 14
 use SAML2\XML\md\SPSSODescriptor;
19 15
 use SAML2\XML\md\SSODescriptorType;
16
+use craft\base\Component;
17
+use flipbox\keychain\records\KeyChainRecord;
18
+use flipbox\saml\core\helpers\SecurityHelper;
19
+use flipbox\saml\core\models\SettingsInterface;
20 20
 use yii\base\Event;
21 21
 use yii\base\InvalidConfigException;
22 22
 
Please login to merge, or discard this patch.
src/validators/Response.php 1 patch
Unused Use Statements   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -3,13 +3,13 @@
 block discarded – undo
3 3
 
4 4
 namespace flipbox\saml\core\validators;
5 5
 
6
-use flipbox\saml\core\records\AbstractProvider;
6
+use SAML2\Assertion\Validation\Result as AssertionResult;
7 7
 use SAML2\Configuration\Destination;
8
+use SAML2\Response as SamlResponse;
8 9
 use SAML2\Response\Validation\ConstraintValidator\DestinationMatches;
9 10
 use SAML2\Response\Validation\ConstraintValidator\IsSuccessful;
10 11
 use SAML2\Response\Validation\Result as ResponseResult;
11
-use SAML2\Assertion\Validation\Result as AssertionResult;
12
-use SAML2\Response as SamlResponse;
12
+use flipbox\saml\core\records\AbstractProvider;
13 13
 
14 14
 class Response
15 15
 {
Please login to merge, or discard this patch.
src/controllers/cp/view/metadata/VariablesTrait.php 1 patch
Unused Use Statements   -2 removed lines patch added patch discarded remove patch
@@ -7,11 +7,9 @@
 block discarded – undo
7 7
 namespace flipbox\saml\core\controllers\cp\view\metadata;
8 8
 
9 9
 use Craft;
10
-use craft\elements\User;
11 10
 use craft\helpers\UrlHelper;
12 11
 use flipbox\keychain\KeyChain;
13 12
 use flipbox\saml\core\AbstractPlugin;
14
-use flipbox\saml\core\helpers\MappingHelper;
15 13
 use flipbox\saml\core\models\SettingsInterface;
16 14
 use flipbox\saml\core\records\ProviderInterface;
17 15
 
Please login to merge, or discard this patch.
src/migrations/m190516_200148_attribute_typo.php 1 patch
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,6 @@
 block discarded – undo
4 4
 
5 5
 use craft\db\Migration;
6 6
 use flipbox\saml\core\records\AbstractProvider;
7
-use yii\db\Query;
8 7
 
9 8
 /**
10 9
  * mm190516_200148_attribute_typo migration.
Please login to merge, or discard this patch.
src/controllers/cp/view/AbstractController.php 2 patches
Doc Comments   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -124,7 +124,6 @@  discard block
 block discarded – undo
124 124
 
125 125
     /**
126 126
      * @param ProviderInterface $provider
127
-     * @param array $variables
128 127
      * @return array
129 128
      */
130 129
     protected function addUrls(ProviderInterface $provider)
@@ -196,7 +195,7 @@  discard block
 block discarded – undo
196 195
     }
197 196
 
198 197
     /**
199
-     * @param $endpoints
198
+     * @param EndpointType[] $endpoints
200 199
      * @return EndpointType|null
201 200
      */
202 201
     protected function getFirstEndpoint($endpoints)
Please login to merge, or discard this patch.
Unused Use Statements   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -8,16 +8,16 @@
 block discarded – undo
8 8
 
9 9
 namespace flipbox\saml\core\controllers\cp\view;
10 10
 
11
+use SAML2\XML\md\EndpointType;
12
+use SAML2\XML\md\IDPSSODescriptor;
13
+use SAML2\XML\md\IndexedEndpointType;
14
+use SAML2\XML\md\SPSSODescriptor;
11 15
 use flipbox\saml\core\AbstractPlugin;
12
-use flipbox\saml\core\controllers\AbstractController as BaseController;
13 16
 use flipbox\saml\core\EnsureSAMLPlugin;
17
+use flipbox\saml\core\controllers\AbstractController as BaseController;
14 18
 use flipbox\saml\core\models\AbstractSettings;
15 19
 use flipbox\saml\core\records\ProviderInterface;
16 20
 use flipbox\saml\core\web\assets\bundles\SamlCore;
17
-use SAML2\XML\md\EndpointType;
18
-use SAML2\XML\md\IDPSSODescriptor;
19
-use SAML2\XML\md\IndexedEndpointType;
20
-use SAML2\XML\md\SPSSODescriptor;
21 21
 
22 22
 /**
23 23
  * Class AbstractController
Please login to merge, or discard this patch.
src/services/bindings/Factory.php 2 patches
Doc Comments   -1 removed lines patch added patch discarded remove patch
@@ -81,7 +81,6 @@
 block discarded – undo
81 81
     }
82 82
 
83 83
     /**
84
-     * @param SamlMessage $message
85 84
      * @param AbstractProvider $provider
86 85
      * @return \SAML2\XML\md\IndexedEndpointType|null
87 86
      */
Please login to merge, or discard this patch.
Unused Use Statements   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -2,18 +2,18 @@
 block discarded – undo
2 2
 
3 3
 namespace flipbox\saml\core\services\bindings;
4 4
 
5
-use craft\base\Component;
6
-use flipbox\saml\core\exceptions\InvalidMetadata;
7
-use flipbox\saml\core\records\AbstractProvider;
8
-use flipbox\saml\core\records\ProviderInterface;
9
-use SAML2\Constants;
10 5
 use SAML2\AuthnRequest;
6
+use SAML2\Constants;
11 7
 use SAML2\HTTPPost;
12 8
 use SAML2\HTTPRedirect;
13 9
 use SAML2\LogoutRequest;
14 10
 use SAML2\LogoutResponse;
15 11
 use SAML2\Message as SamlMessage;
16 12
 use SAML2\Response;
13
+use craft\base\Component;
14
+use flipbox\saml\core\exceptions\InvalidMetadata;
15
+use flipbox\saml\core\records\AbstractProvider;
16
+use flipbox\saml\core\records\ProviderInterface;
17 17
 
18 18
 /**
19 19
  * Class Factory
Please login to merge, or discard this patch.
src/services/messages/LogoutResponse.php 1 patch
Unused Use Statements   +2 added lines, -3 removed lines patch added patch discarded remove patch
@@ -6,12 +6,11 @@
 block discarded – undo
6 6
 
7 7
 namespace flipbox\saml\core\services\messages;
8 8
 
9
+use SAML2\LogoutRequest;
10
+use SAML2\LogoutResponse as SamlLogoutResponse;
9 11
 use craft\base\Component;
10 12
 use flipbox\saml\core\models\SettingsInterface;
11 13
 use flipbox\saml\core\records\AbstractProvider;
12
-use SAML2\Constants;
13
-use SAML2\LogoutRequest;
14
-use SAML2\LogoutResponse as SamlLogoutResponse;
15 14
 use yii\base\Event;
16 15
 
17 16
 /**
Please login to merge, or discard this patch.
src/AbstractPlugin.php 1 patch
Unused Use Statements   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -7,25 +7,25 @@
 block discarded – undo
7 7
 
8 8
 namespace flipbox\saml\core;
9 9
 
10
+use SAML2\Compat\AbstractContainer;
10 11
 use craft\base\Plugin;
11 12
 use craft\events\RegisterTemplateRootsEvent;
12 13
 use craft\events\RegisterUrlRulesEvent;
13 14
 use craft\helpers\StringHelper;
14 15
 use craft\helpers\UrlHelper;
15
-use craft\web\twig\variables\CraftVariable;
16 16
 use craft\web\View;
17
+use craft\web\twig\variables\CraftVariable;
17 18
 use flipbox\craft\psr3\Logger;
18 19
 use flipbox\saml\core\models\AbstractSettings;
19 20
 use flipbox\saml\core\models\SettingsInterface;
20 21
 use flipbox\saml\core\services\AbstractCp;
21
-use flipbox\saml\core\services\bindings\Factory;
22 22
 use flipbox\saml\core\services\Cp;
23
-use flipbox\saml\core\services\messages\LogoutRequest;
24
-use flipbox\saml\core\services\messages\LogoutResponse;
25 23
 use flipbox\saml\core\services\Metadata;
26 24
 use flipbox\saml\core\services\ProviderIdentityServiceInterface;
27 25
 use flipbox\saml\core\services\ProviderServiceInterface;
28
-use SAML2\Compat\AbstractContainer;
26
+use flipbox\saml\core\services\bindings\Factory;
27
+use flipbox\saml\core\services\messages\LogoutRequest;
28
+use flipbox\saml\core\services\messages\LogoutResponse;
29 29
 use yii\base\Event;
30 30
 
31 31
 /**
Please login to merge, or discard this patch.
src/models/AbstractSettings.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -152,6 +152,9 @@
 block discarded – undo
152 152
         return $this->getMyType() === self::SP;
153 153
     }
154 154
 
155
+    /**
156
+     * @param string $url
157
+     */
155 158
     protected function buildEndpointUrl($url)
156 159
     {
157 160
         return sprintf('/%s/%s', $this->getEndpointPrefix(), $url);
Please login to merge, or discard this patch.