modifications.

modifications.

Author: cristian-vatca-osf

Api/GigyaAccountRepositoryInterface.php

@@ -22,13 +22,13 @@ interface GigyaAccountRepositoryInterface
      * @throws GSApiException If error encountered on service call or functional error returned by service.
      * Check error code to identify the case.
      */
-    public function update($gigyaAccount);
+    public function update(GigyaUser $gigyaAccount);
 
     /**
      * Get a Gigya customer account.
      *
      * @param string $uid
      * @return GigyaUser
      */
-    public function get($uid);
+    public function get(string $uid): GigyaUser;
 }

Api/GigyaAccountServiceInterface.php

@@ -33,15 +33,15 @@ interface GigyaAccountServiceInterface
      * @throws GSApiException If error encountered on service call or functional error returned by service.
      * Check error code to identify the case.
      */
-    public function update($gigyaAccount);
+    public function update(GigyaUser $gigyaAccount);
 
     /**
      * Get a Gigya customer account.
      *
      * @param string $uid
      * @return GigyaUser
      */
-    public function get($uid);
+    public function get(string $uid);
 
     /**
      * Update the Gigya service with the latest successfully updated version of an account.
@@ -51,5 +51,5 @@ public function get($uid);
      * @param $uid string
      * @return GigyaUser null if the rollback failed or if the given uid has not been updated so far (thus nothing to roll back)
      */
-    public function rollback($uid);
+    public function rollback(string $uid);
 }

Block/Account/Dashboard/GigyaInfo.php

@@ -49,15 +49,15 @@ public function __construct(
     /**
      * @return string
      */
-    public function getProfileDesktopScreensetId()
+    public function getProfileDesktopScreensetId(): string
     {
         return $this->configModel->getProfileDesktopScreensetId();
     }
 
     /**
      * @return string
      */
-    public function getProfileMobileScreensetId()
+    public function getProfileMobileScreensetId(): string
     {
         return $this->configModel->getProfileMobileScreensetId();
     }

Block/Adminhtml/Customer/Edit/Tab/View/GigyaUID.php

@@ -2,16 +2,18 @@
 
 namespace Gigya\GigyaIM\Block\Adminhtml\Customer\Edit\Tab\View;
 
+use Magento\Backend\Block\Template;
+use Magento\Customer\Api\Data\CustomerInterface;
 use Magento\Customer\Block\Adminhtml\Edit\Tab\View\PersonalInfo;
 
-class GigyaUID extends \Magento\Backend\Block\Template
+class GigyaUID extends Template
 {
     /**
      * Extract the Gigya ID from the customer present in the parent block.
      *
      * @return bool|string
      */
-    public function getGUIDFromParentBlock()
+    public function getGUIDFromParentBlock(): bool|string
     {
         $customer = $this->getCustomerFromParentBlock();
         if ($customer) {
@@ -27,9 +29,9 @@ public function getGUIDFromParentBlock()
     /**
      * Extract the current customer from the parent block.
      *
-     * @return bool|\Magento\Customer\Api\Data\CustomerInterface
+     * @return bool|CustomerInterface
      */
-    protected function getCustomerFromParentBlock()
+    protected function getCustomerFromParentBlock(): bool|CustomerInterface
     {
         $parentBlock = $this->getParentBlock();
         if ($parentBlock && $parentBlock instanceof PersonalInfo) {

Block/Adminhtml/Form/AdditionalScreenSet.php

@@ -8,7 +8,7 @@
 
 class AdditionalScreenSet extends AbstractFieldArray
 {
-    private $_checkboxRenderer;
+    private Checkbox $_checkboxRenderer;
 
     public function __construct(Context $context, Checkbox $checkboxRenderer)
     {
@@ -20,7 +20,7 @@ public function __construct(Context $context, Checkbox $checkboxRenderer)
     /**
      * {@inheritdoc}
      */
-    protected function _prepareToRender()
+    protected function _prepareToRender(): void
     {
         $this->addColumn('desktop_screen', ['label' => __('Screen-Set ID'), 'class' => 'required-entry', 'renderer' => false]);
         $this->addColumn('mobile_screen', ['label' => __('Mobile Screen-Set ID'), 'renderer' => false]);

Block/Adminhtml/Form/EmptyTextarea.php

@@ -7,7 +7,7 @@
 
 class EmptyTextarea extends Field
 {
-    protected function _getElementHtml(AbstractElement $element)
+    protected function _getElementHtml(AbstractElement $element): string
     {
         $value = $element->getEscapedValue();
         $label = $element->getLabel() ?? 'Value';

Block/Adminhtml/System/Config/View/Checkbox.php

@@ -6,7 +6,7 @@
 
 class Checkbox extends AbstractBlock
 {
-    protected function _toHtml()
+    protected function _toHtml(): string
     {
         $elementId   = $this->getInputId();
         $elementName = $this->getInputName();

Block/Form/GigyaAuthenticationPopup.php

@@ -6,6 +6,7 @@
 namespace Gigya\GigyaIM\Block\Form;
 
 use \Magento\Customer\Block\Account\AuthenticationPopup;
+use Magento\Framework\Exception\LocalizedException;
 use \Magento\Framework\View\Element\Template\Context;
 use \Magento\Framework\Serialize\Serializer\Json;
 use \Gigya\GigyaIM\Model\Config;
@@ -20,7 +21,7 @@ class GigyaAuthenticationPopup extends AuthenticationPopup
     /**
      * @var Config
      */
-    protected $config;
+    protected Config $config;
 
     /**
      * GigyaAuthenticationPopup constructor.
@@ -43,23 +44,24 @@ public function __construct(
     /**
      * @return string
      */
-    public function getLoginDesktopScreensetId()
+    public function getLoginDesktopScreensetId(): string
     {
         return $this->config->getLoginDesktopScreensetId();
     }
 
     /**
      * @return string
      */
-    public function getLoginMobileScreensetId()
+    public function getLoginMobileScreensetId(): string
     {
         return $this->config->getLoginMobileScreensetId();
     }
 
     /**
      * @return string
+     * @throws LocalizedException
      */
-    public function _toHtml()
+    public function _toHtml(): string
     {
 
         if ($this->config->isGigyaEnabled()) {

Block/Form/GigyaLogin.php

@@ -22,7 +22,7 @@ class GigyaLogin extends Login
     /**
      * @var Config
      */
-    protected $configModel;
+    protected Config $configModel;
 
     /**
      * Login constructor.
@@ -47,25 +47,25 @@ public function __construct(
     /**
      * @return string
      */
-    public function getLoginDesktopScreensetId()
+    public function getLoginDesktopScreensetId(): string
     {
-        return $this->configModel->getLoginDesktopScreensetId();
+        return (string)$this->configModel->getLoginDesktopScreensetId();
     }
 
     /**
      * @return string
      */
-    public function getLoginMobileScreensetId()
+    public function getLoginMobileScreensetId(): string
     {
-        return $this->configModel->getLoginMobileScreensetId();
+        return (string)$this->configModel->getLoginMobileScreensetId();
     }
 
     /**
      * @return string
      *
      * @throws LocalizedException
      */
-    public function _toHtml()
+    public function _toHtml(): string
     {
         if ($this->configModel->isGigyaEnabled()) {
             $this->getLayout()->unsetElement('customer.login.container');

Block/Form/GigyaRegister.php

@@ -27,7 +27,7 @@ class GigyaRegister extends Register
     /**
      * @var GigyaConfig
      */
-    protected $configModel;
+    protected GigyaConfig $configModel;
 
     /**
      * Register constructor.
@@ -75,15 +75,15 @@ public function __construct(
     /**
      * @return string
      */
-    public function getLoginDesktopScreensetId()
+    public function getLoginDesktopScreensetId(): string
     {
         return $this->configModel->getLoginDesktopScreensetId();
     }
 
     /**
      * @return string
      */
-    public function getLoginMobileScreensetId()
+    public function getLoginMobileScreensetId(): string
     {
         return $this->configModel->getLoginMobileScreensetId();
     }