[3][com_postinstall] warning on php 8

[alikon]

Pull Request for Issue #31483 .

Summary of Changes

php 8 compatibility

Testing Instructions

use php 8
go to Admin->Control Panel

Actual result BEFORE applying this Pull Request

Warning: FOFModelBehavior::onBeforeBuildQuery(): Argument #1 ($model) must be passed by reference, value given in \libraries\fof\utils\observable\event.php on line 63

Expected result AFTER applying this Pull Request

no warning

[810]

I have tested this item ✅ successfully on e050d05

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31537.}

[ghost]

I have tested this item ✅ successfully on e050d05

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31537.}

[alikon]

RTC

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31537.}

[jameswadsworth]

I have tested this item ✅ successfully on e050d05

This resolves the issue in php 7.4

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/31537.}

[HLeithner]

can you please add the modified doc block to both files?

* @note This file has been modified by the Joomla! Project and no longer reflects the original work of its author.

[alikon]

yes, done

[HLeithner]

Thanks

[SharkyKZ]

This shouldn't have been merged.

[Fedik]

@SharkyKZ what alternative for this can be?

[Fedik]

but I agree with @SharkyKZ , the changes here will lead to Fatal Error to whose extend the class, with error
Declaration of ' ' must be compatible with ' '

[Fedik]

I have revert it, and made a better fix, please test #31600

[HLeithner]

@810 can you test the new version of this pr at #31600 please?