=Switched the /base/options.php to the same pattern as Site.php: a class ba...

Jake Vanderwerf

yesterday f4be611c51473359e6d41780f0313c446079e9d3

 inc/registrar/config/Integration.php

@@ -13,7 +13,7 @@
    * @var string Must match as defined in the JVBase\integrations namespace
    */
   protected string $service_name;
   protected string $content_type;
   protected ?string $content_type = null;
   /**
    * @var bool Whether to send to the integration on publish
    */
@@ -38,18 +38,27 @@
      $this->service_name = $service;
   }


   public function getService_name():string
   {
      return $this->service_name;
   }

   /**
    * @param string $content must match what integration expects
    * @param ?string $content must match what integration expects
    * @return self
    */
   public function setContent_type(string $content):self
   public function setContentType(?string $content):self
   {
      $allowed = JVB()->connect($this->service_name)->allowedContent();
      if (is_null($content)) {
         return $this;
      }
      $connection = JVB()->connect($this->service_name);
      if (!$connection){
         error_log('[Integration]::setContentType Service is not setup. '.$this->service_name);
         return $this;
      }
      $allowed = $connection->getAllowedContent();
      if (!in_array($content, $allowed)) {
         error_log($this->service_name.' Connection does not support this content: '.$content);
         return $this;
@@ -58,7 +67,7 @@
      return $this;
   }

   public function getContentType():string
   public function getContentType():?string
   {
      return $this->content_type;
   }