N°1986 Portal: Forms with only transition buttons are now read-only

datamodels/2.x/itop-portal-base/portal/src/Form/ObjectFormManager.php

@@ -82,6 +82,8 @@ class ObjectFormManager extends FormManager
 	protected $aFormProperties;
 	/** @var array $aCallbackUrls */
 	protected $aCallbackUrls = array();
+	/** @var boolean $bIsSubmittable */
+	protected $bIsSubmittable = true;
 	
 	/**
 	 * Creates an instance of \Combodo\iTop\Portal\Form\ObjectFormManager from JSON data that must contain at least :
@@ -228,6 +230,28 @@ class ObjectFormManager extends FormManager
 		return $this;
 	}
 	
+	/**
+	 *
+	 * @return string
+	 */
+	public function GetIsSubmittable()
+	{
+		return $this->bIsSubmittable;
+	}
+
+	/**
+	 *
+	 * @param boolean $bIsSubmittable
+	 *
+	 * @return \Combodo\iTop\Portal\Form\ObjectFormManager
+	 */
+	public function SetIsSubmittable($bIsSubmittable)
+	{
+		$this->bIsSubmittable = $bIsSubmittable;
+
+		return $this;
+	}
+	
 	/**
 	 *
 	 * @return string
@@ -462,6 +486,12 @@ class ObjectFormManager extends FormManager
 					continue;
 				}
 				
+				// If a form can't be submitted to update an object, we have no reason to allow fields to be editable
+				if (!$this->GetIsSubmittable())
+				{
+					$sFieldFlags .= ' read_only';
+				}
+
 				// Otherwise we proceed as usual
 				foreach (explode(' ', $sFieldFlags) as $sFieldFlag)
 				{

datamodels/2.x/itop-portal-base/portal/src/Helper/ObjectFormHandlerHelper.php

@@ -246,7 +246,8 @@ class ObjectFormHandlerHelper
 				->SetMode($sMode)
 				->SetActionRulesToken($sActionRulesToken)
 				->SetRenderer($oFormRenderer)
-				->SetFormProperties($aFormProperties);
+				->SetFormProperties($aFormProperties)
+				->SetIsSubmittable(isset($aFormData['buttons']['submit']));
 
 			$oFormManager->Build();