[skip ci] Update compile method signatures for PHP 8 compatibility

jcheron · jcheron · commit 1c975551c5d1 · 2025-11-30T02:14:19.000+01:00
Changed the signatures of various compile methods to use PHP 8 union and nullable types, replacing 'JsUtils $js = NULL, &amp;$view = NULL' with '?JsUtils $js = NULL, mixed &amp;$view = NULL' where appropriate. This improves type safety and compatibility with newer PHP versions.
diff --git a/Ajax/bootstrap/html/HtmlAlert.php b/Ajax/bootstrap/html/HtmlAlert.php
@@ -78,7 +78,7 @@ public function close() {
 	 * (non-PHPdoc)
 	 * @see \Ajax\bootstrap\html\base\BaseHtml::compile()
 	 */
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {
 		if ($this->closeable&&$this->button==="") {
 			$this->addCloseButton();
 		}
diff --git a/Ajax/bootstrap/html/HtmlBreadcrumbs.php b/Ajax/bootstrap/html/HtmlBreadcrumbs.php
@@ -123,7 +123,7 @@ public function getHref($index=null,$separator="/"){
 	 * (non-PHPdoc)
 	 * @see \Ajax\bootstrap\html\BaseHtml::compile()
 	 */
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {
 		if($this->autoActive){
 			$this->setActive();
 		}
diff --git a/Ajax/bootstrap/html/HtmlCarousel.php b/Ajax/bootstrap/html/HtmlCarousel.php
@@ -141,7 +141,7 @@ private function createIndicator() {
 	 * (non-PHPdoc)
 	 * @see \Ajax\bootstrap\html\base\BaseHtml::compile()
 	 */
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {
 		$this->slides [0]->setClass("item active");
 		$this->indicators [0]->setClass("active");
 		$this->createControls();
diff --git a/Ajax/bootstrap/html/HtmlForm.php b/Ajax/bootstrap/html/HtmlForm.php
@@ -24,7 +24,7 @@ public function __construct($identifier) {
 	 * (non-PHPdoc)
 	 * @see \Ajax\bootstrap\html\base\BaseHtml::compile()
 	 */
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {
 		if (isset($js)) {
 			$this->formElementsPrefix=$js->config()->getVar("formElementsPrefix");
 			foreach ( $this->futureElements as $futureElement ) {
diff --git a/Ajax/bootstrap/html/HtmlGridSystem.php b/Ajax/bootstrap/html/HtmlGridSystem.php
@@ -90,7 +90,7 @@ public function getCellAt($row,$col,$force=true){
 		return $col;
 	}
 	
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {
 		foreach ($this->rows as $row){
 			$this->addContent($row);
 		}
diff --git a/Ajax/bootstrap/html/HtmlProgressbar.php b/Ajax/bootstrap/html/HtmlProgressbar.php
@@ -132,7 +132,7 @@ public function setStyle($cssStyle) {
 	 * (non-PHPdoc)
 	 * @see \Ajax\bootstrap\html\base\BaseHtml::compile()
 	 */
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {
 		$actualStyle=$this->style;
 		if(isset($this->styleLimits)&& JArray::isAssociative($this->styleLimits)){
 			foreach ($this->styleLimits as $k=>$v){
diff --git a/Ajax/bootstrap/html/HtmlTabs.php b/Ajax/bootstrap/html/HtmlTabs.php
@@ -95,7 +95,7 @@ public function setTabstype($_tabsType="tabs") {
 	 * (non-PHPdoc)
 	 * @see \Ajax\bootstrap\html\BaseHtml::compile()
 	 */
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {
 		$this->setProperty("class", "nav nav-".$this->_tabsType." ".$this->stacked);
 		return parent::compile($js, $view);
 	}
diff --git a/Ajax/bootstrap/html/content/HtmlGridCol.php b/Ajax/bootstrap/html/content/HtmlGridCol.php
@@ -36,7 +36,7 @@ private function _generateClass(){
 		return implode(" ", $result);
 	}
 
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {
 		$this->setProperty("class", $this->_generateClass());
 		return parent::compile($js,$view);
 	}
diff --git a/Ajax/bootstrap/html/content/HtmlGridRow.php b/Ajax/bootstrap/html/content/HtmlGridRow.php
@@ -63,7 +63,7 @@ public function getColAt($offset,$force=true){
 		return $result;
 	}
 
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {
 
 		foreach ($this->cols as $col){
 			$this->addContent($col);
diff --git a/Ajax/bootstrap/html/content/HtmlListgroupItem.php b/Ajax/bootstrap/html/content/HtmlListgroupItem.php
@@ -54,7 +54,7 @@ public function __toString() {
 	 * (non-PHPdoc)
 	 * @see \Ajax\bootstrap\html\base\BaseHtml::compile()
 	 */
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {
 		return $this->element->compile($js, $view);
 	}
 
diff --git a/Ajax/common/components/BaseComponent.php b/Ajax/common/components/BaseComponent.php
@@ -48,7 +48,7 @@ public function getParams() {
 		return $this->params;
 	}
 
-	public function compile(JsUtils $js=NULL) {
+	public function compile(?JsUtils $js=NULL) {
 		if ($js==NULL)
 			$js=$this->js;
 		$script=$this->getScript();
diff --git a/Ajax/common/html/BaseHtml.php b/Ajax/common/html/BaseHtml.php
@@ -205,7 +205,7 @@ protected function compile_once(JsUtils $js = NULL, &$view = NULL) {
 		}
 	}
 
-	public function compile(JsUtils $js = NULL, &$view = NULL) {
+	public function compile(?JsUtils $js = NULL, mixed &$view = NULL) {
 		$this->compile_once($js, $view);
 		$result = $this->getTemplate($js, $view);
 		foreach ($this as $key => $value) {
diff --git a/Ajax/common/html/html5/HtmlSelect.php b/Ajax/common/html/html5/HtmlSelect.php
@@ -48,7 +48,7 @@ public function setValue($value) {
 	 * (non-PHPdoc)
 	 * @see \Ajax\bootstrap\html\base\BaseHtml::compile()
 	 */
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {
 		$this->content=array();
 		if(isset($this->default)){
 			$default=new HtmlOption("", $this->default);
diff --git a/Ajax/semantic/html/base/HtmlSemDoubleElement.php b/Ajax/semantic/html/base/HtmlSemDoubleElement.php
@@ -166,7 +166,7 @@ public function jsShowDimmer($show = true) {
 	 * {@inheritdoc}
 	 * @see BaseHtml::compile()
 	 */
-	public function compile(JsUtils $js = NULL, &$view = NULL) {
+	public function compile(?JsUtils $js = NULL, mixed &$view = NULL) {
 		if (isset($this->_popup)) {
 			$this->_popup->compile($js);
 		}
diff --git a/Ajax/semantic/html/collections/HtmlBreadcrumb.php b/Ajax/semantic/html/collections/HtmlBreadcrumb.php
@@ -156,7 +156,7 @@ public function setAutoActive($autoActive) {
 	 * (non-PHPdoc)
 	 * @see \Ajax\bootstrap\html\BaseHtml::compile()
 	 */
-	public function compile(JsUtils $js = NULL, &$view = NULL) {
+	public function compile(?JsUtils $js = NULL, mixed &$view = NULL) {
 		if ($this->autoActive) {
 			$this->setActive();
 		}
diff --git a/Ajax/semantic/html/collections/menus/HtmlPaginationMenu.php b/Ajax/semantic/html/collections/menus/HtmlPaginationMenu.php
@@ -18,7 +18,7 @@ public function __construct( $identifier, $items=array() ){
 	 * {@inheritDoc}
 	 * @see \Ajax\common\html\BaseHtml::compile()
 	 */
-	public function compile(JsUtils $js=NULL,&$view=NULL){
+	public function compile(?JsUtils $js=NULL,mixed &$view=NULL){
 		$max=$this->_max;
 		if(!$this->_compiled){
 			foreach ($this->content as $item){
diff --git a/Ajax/semantic/html/collections/table/HtmlTable.php b/Ajax/semantic/html/collections/table/HtmlTable.php
@@ -414,7 +414,7 @@ public function applyRows($callback) {
 	 *
 	 * @see HtmlSemDoubleElement::compile()
 	 */
-	public function compile(JsUtils $js = NULL, &$view = NULL) {
+	public function compile(?JsUtils $js = NULL, mixed &$view = NULL) {
 		if (\sizeof($this->_compileParts) < 3) {
 			$this->_template = "%content%";
 			$this->refresh($js);
diff --git a/Ajax/semantic/html/content/HtmlGridRow.php b/Ajax/semantic/html/content/HtmlGridRow.php
@@ -123,7 +123,7 @@ protected function createItem($value){
 		return $col;
 	}
 
-	public function compile(JsUtils $js=NULL,&$view=NULL){
+	public function compile(?JsUtils $js=NULL,mixed &$view=NULL){
 		if($this->_implicite===true){
 			$this->_template="%wrapContentBefore%%content%%wrapContentAfter%";
 		}
diff --git a/Ajax/semantic/html/content/InternalPopup.php b/Ajax/semantic/html/content/InternalPopup.php
@@ -35,7 +35,7 @@ public function onShow($jsCode){
 		$this->params["onShow"]=$jsCode;
 	}
 
-	public function compile(JsUtils $js=NULL){
+	public function compile(?JsUtils $js=NULL){
 		if(JString::isNotNull($this->title)){
 			$this->semElement->setProperty("data-title", $this->title);
 		}
diff --git a/Ajax/semantic/html/content/table/HtmlTD.php b/Ajax/semantic/html/content/table/HtmlTD.php
@@ -154,7 +154,7 @@ public function toDelete() {
 		return $this;
 	}
 
-	public function compile(JsUtils $js = NULL, &$view = NULL) {
+	public function compile(?JsUtils $js = NULL, mixed &$view = NULL) {
 		if (! $this->_deleted)
 			return parent::compile($js, $view);
 	}
diff --git a/Ajax/semantic/html/content/view/HtmlViewContent.php b/Ajax/semantic/html/content/view/HtmlViewContent.php
@@ -139,7 +139,7 @@ public function addHeaderContent($header, $metas=array(), $description=NULL,$ext
 		return $this;
 	}
 
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {
 		return parent::compile($js, $view);
 	}
 }
diff --git a/Ajax/semantic/html/content/view/HtmlViewItem.php b/Ajax/semantic/html/content/view/HtmlViewItem.php
@@ -140,7 +140,7 @@ public function getItemHeader() {
 	 *
 	 * @see HtmlSemDoubleElement::compile()
 	 */
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {
 		$this->content=JArray::sortAssociative($this->content, ["header","image","icon","content","extra-content"]);
 		return parent::compile($js, $view);
 	}
diff --git a/Ajax/semantic/html/elements/HtmlList.php b/Ajax/semantic/html/elements/HtmlList.php
@@ -150,7 +150,7 @@ public function setHorizontal() {
 	 * {@inheritdoc}
 	 * @see \Ajax\common\html\HtmlCollection::compile()
 	 */
-	public function compile(JsUtils $js = NULL, &$view = NULL) {
+	public function compile(?JsUtils $js = NULL, mixed &$view = NULL) {
 		if (isset($this->_maxVisible) && $this->_maxVisible < $this->count()) {
 			$this->addFollowPoints();
 			if (isset($js)) {
diff --git a/Ajax/semantic/html/modules/HtmlDropdown.php b/Ajax/semantic/html/modules/HtmlDropdown.php
@@ -421,7 +421,7 @@ public function setFullTextSearch($value) {
 		return $this;
 	}
 
-	public function compile(JsUtils $js = NULL, &$view = NULL) {
+	public function compile(?JsUtils $js = NULL, mixed &$view = NULL) {
 		$this->applyValue();
 		return parent::compile($js, $view);
 	}
diff --git a/Ajax/semantic/html/modules/HtmlProgress.php b/Ajax/semantic/html/modules/HtmlProgress.php
@@ -62,7 +62,7 @@ public function setError() {
 	 *
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {
 		$this->content=JArray::sortAssociative($this->content, [ "bar","label" ]);
 		return parent::compile($js, $view);
 	}
diff --git a/Ajax/semantic/html/modules/HtmlTab.php b/Ajax/semantic/html/modules/HtmlTab.php
@@ -287,7 +287,7 @@ public function run(JsUtils $js) {
 		return $this->_bsComponent;
 	}
 
-	public function compile(JsUtils $js = NULL, &$view = NULL) {
+	public function compile(?JsUtils $js = NULL, mixed &$view = NULL) {
 		if (! $this->_activated && $this->content["menu"]->count() > 0 && \sizeof($this->content) > 1)
 			$this->activate(0);
 		return parent::compile($js, $view);
diff --git a/Ajax/semantic/html/modules/HtmlToast.php b/Ajax/semantic/html/modules/HtmlToast.php
@@ -35,7 +35,7 @@ public function setContent($value) {
 	 *
 	 * @see \Ajax\semantic\html\base\HtmlSemDoubleElement::compile()
 	 */
-	public function compile(JsUtils $js=NULL, &$view=NULL) {
+	public function compile(?JsUtils $js=NULL,mixed &$view=NULL) {
 		$this->content=JArray::sortAssociative($this->content, ['content','actions' ]);
 		return parent::compile($js, $view);
 	}
diff --git a/Ajax/semantic/widgets/business/BusinessForm.php b/Ajax/semantic/widgets/business/BusinessForm.php
@@ -53,7 +53,7 @@ public function removeField($fieldName){
 		return $this;
 	}
 
-	public function compile(JsUtils $js=NULL,&$view=NULL){
+	public function compile(?JsUtils $js=NULL,mixed &$view=NULL){
 		return parent::compile($js,$view);
 	}
 }
diff --git a/Ajax/semantic/widgets/dataform/DataForm.php b/Ajax/semantic/widgets/dataform/DataForm.php
@@ -33,7 +33,7 @@ protected function _getFieldIdentifier($prefix,$name=''){
 		return $this->identifier."-{$name}-".$this->_instanceViewer->getIdentifier();
 	}
 
-	public function compile(JsUtils $js=NULL,&$view=NULL){
+	public function compile(?JsUtils $js=NULL,mixed &$view=NULL){
 		if(!$this->_generated){
 			$this->_instanceViewer->setInstance($this->_modelInstance);
 
diff --git a/Ajax/semantic/widgets/datatable/DataTable.php b/Ajax/semantic/widgets/datatable/DataTable.php
@@ -136,7 +136,7 @@ public function refreshTD($fieldName, $jquery, $view) {
 			->getCell(0, $index);
 	}
 
-	public function compile(JsUtils $js = NULL, &$view = NULL) {
+	public function compile(?JsUtils $js = NULL,mixed &$view = NULL) {
 		if (! $this->_generated) {
 			if (isset($this->_buttonsColumn)) {
 				$this->_instanceViewer->sortColumnContent($this->_buttonsColumn, $this->_buttons);
diff --git a/Ajax/service/AjaxCall.php b/Ajax/service/AjaxCall.php
@@ -14,7 +14,7 @@ public function __construct($method, $parameters) {
 		$this->parameters = $parameters;
 	}
 
-	public function compile(JsUtils $js = null) {
+	public function compile(?JsUtils $js = null) {
 		if ($js === null)
 			return;
 		$params = "{}";

Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ public function close() {`
`78`	`78`	`* (non-PHPdoc)`
`79`	`79`	`* @see \Ajax\bootstrap\html\base\BaseHtml::compile()`
`80`	`80`	`*/`
`81`		`- public function compile(JsUtils $js=NULL, &$view=NULL) {`
	`81`	`+ public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {`
`82`	`82`	`if ($this->closeable&&$this->button==="") {`
`83`	`83`	`$this->addCloseButton();`
`84`	`84`	`}`
Original file line number	Diff line number	Diff line change
`@@ -123,7 +123,7 @@ public function getHref($index=null,$separator="/"){`
`123`	`123`	`* (non-PHPdoc)`
`124`	`124`	`* @see \Ajax\bootstrap\html\BaseHtml::compile()`
`125`	`125`	`*/`
`126`		`- public function compile(JsUtils $js=NULL, &$view=NULL) {`
	`126`	`+ public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {`
`127`	`127`	`if($this->autoActive){`
`128`	`128`	`$this->setActive();`
`129`	`129`	`}`
Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,7 @@ public function getCellAt($row,$col,$force=true){`
`90`	`90`	`return $col;`
`91`	`91`	`}`
`92`	`92`
`93`		`- public function compile(JsUtils $js=NULL, &$view=NULL) {`
	`93`	`+ public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {`
`94`	`94`	`foreach ($this->rows as $row){`
`95`	`95`	`$this->addContent($row);`
`96`	`96`	`}`
Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,7 @@ public function setTabstype($_tabsType="tabs") {`
`95`	`95`	`* (non-PHPdoc)`
`96`	`96`	`* @see \Ajax\bootstrap\html\BaseHtml::compile()`
`97`	`97`	`*/`
`98`		`- public function compile(JsUtils $js=NULL, &$view=NULL) {`
	`98`	`+ public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {`
`99`	`99`	`$this->setProperty("class", "nav nav-".$this->_tabsType." ".$this->stacked);`
`100`	`100`	`return parent::compile($js, $view);`
`101`	`101`	`}`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ private function _generateClass(){`
`36`	`36`	`return implode(" ", $result);`
`37`	`37`	`}`
`38`	`38`
`39`		`- public function compile(JsUtils $js=NULL, &$view=NULL) {`
	`39`	`+ public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {`
`40`	`40`	`$this->setProperty("class", $this->_generateClass());`
`41`	`41`	`return parent::compile($js,$view);`
`42`	`42`	`}`
Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,7 @@ public function getColAt($offset,$force=true){`
`63`	`63`	`return $result;`
`64`	`64`	`}`
`65`	`65`
`66`		`- public function compile(JsUtils $js=NULL, &$view=NULL) {`
	`66`	`+ public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {`
`67`	`67`
`68`	`68`	`foreach ($this->cols as $col){`
`69`	`69`	`$this->addContent($col);`
Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@ public function __toString() {`
`54`	`54`	`* (non-PHPdoc)`
`55`	`55`	`* @see \Ajax\bootstrap\html\base\BaseHtml::compile()`
`56`	`56`	`*/`
`57`		`- public function compile(JsUtils $js=NULL, &$view=NULL) {`
	`57`	`+ public function compile(?JsUtils $js=NULL, mixed &$view=NULL) {`
`58`	`58`	`return $this->element->compile($js, $view);`
`59`	`59`	`}`
`60`	`60`