[7.0] Remove usage of LegacyPropertyManagementTrait from Table class

administrator/components/com_finder/src/Indexer/Taxonomy.php

@@ -13,6 +13,7 @@
 use Joomla\CMS\Factory;
 use Joomla\CMS\Tree\NodeInterface;
 use Joomla\Component\Finder\Administrator\Table\MapTable;
+use Joomla\Utilities\ArrayHelper;
 
 // phpcs:disable PSR1.Files.SideEffects
 \defined('_JEXEC') or die;
@@ -256,7 +257,7 @@ protected static function storeNode($node, $parentId)
         $nodeTable->rebuildPath($nodeTable->id);
 
         // Add the node to the cache.
-        static::$nodes[$parentId . ':' . $nodeTable->title] = (object) $nodeTable->getProperties();
+        static::$nodes[$parentId . ':' . $nodeTable->title] = (object) ArrayHelper::fromObject($nodeTable, false);
 
         return static::$nodes[$parentId . ':' . $nodeTable->title]->id;
     }

administrator/components/com_finder/src/Table/MapTable.php

@@ -16,6 +16,7 @@
 use Joomla\CMS\Table\Nested;
 use Joomla\Database\DatabaseInterface;
 use Joomla\Event\DispatcherInterface;
+use Joomla\Utilities\ArrayHelper;
 
 // phpcs:disable PSR1.Files.SideEffects
 \defined('_JEXEC') or die;
@@ -72,7 +73,7 @@ public function check()
         $this->alias = ApplicationHelper::stringURLSafe($this->title, $this->language);
 
         if (trim($this->alias) == '') {
-            $this->alias = md5(serialize($this->getProperties()));
+            $this->alias = md5(serialize(ArrayHelper::fromObject($this, false)));
         }
 
         return true;

administrator/components/com_guidedtours/src/Model/TourModel.php

@@ -221,7 +221,7 @@ public function getItem($pk = null)
         }
 
         // Convert to an object before adding other data.
-        $properties = $table->getProperties(1);
+        $properties = ArrayHelper::fromObject($table, false);
         $item       = ArrayHelper::toObject($properties);
 
         if (property_exists($item, 'params')) {

administrator/components/com_languages/src/Model/LanguageModel.php

@@ -129,7 +129,7 @@ public function getItem($langId = null)
             $table->access = (int) Factory::getApplication()->get('access');
         }
 
-        $properties = $table->getProperties(1);
+        $properties = ArrayHelper::fromObject($table, false);
         $value      = ArrayHelper::toObject($properties);
 
         return $value;

administrator/components/com_mails/src/Model/TemplateModel.php

@@ -163,7 +163,7 @@ public function getItem($pk = null)
         }
 
         // Convert to an object before adding other data.
-        $properties = $table->getProperties(1);
+        $properties = ArrayHelper::fromObject($table, false);
         $item       = ArrayHelper::toObject($properties);
 
         if (property_exists($item, 'params')) {
@@ -209,7 +209,7 @@ public function getMaster($pk = null)
         }
 
         // Convert to an object before adding other data.
-        $properties = $table->getProperties(1);
+        $properties = ArrayHelper::fromObject($table, false);
         $item       = ArrayHelper::toObject($properties);
 
         if (property_exists($item, 'params')) {

administrator/components/com_menus/src/Model/ItemModel.php

@@ -715,7 +715,7 @@ public function getItem($pk = null)
         $this->setState('item.type', $table->type);
 
         // Convert to the \Joomla\CMS\Object\CMSObject before adding the params.
-        $properties = $table->getProperties(1);
+        $properties = ArrayHelper::fromObject($table, false);
         $result     = ArrayHelper::toObject($properties);
 
         // Convert the params field to an array.
@@ -1024,7 +1024,7 @@ protected function getMenuType($menutype)
 
         $table->load(['menutype' => $menutype]);
 
-        return (object) $table->getProperties();
+        return (object) ArrayHelper::fromObject($table, false);
     }
 
     /**

administrator/components/com_menus/src/Model/MenuModel.php

@@ -145,7 +145,7 @@ public function getItem($itemId = null)
             return false;
         }
 
-        $properties = $table->getProperties(1);
+        $properties = ArrayHelper::fromObject($table, false);
         $value      = ArrayHelper::toObject($properties);
 
         return $value;

administrator/components/com_modules/src/Model/ModuleModel.php

@@ -704,7 +704,7 @@ public function getItem($pk = null)
             }
 
             // Convert to an object before adding other data.
-            $properties        = $table->getProperties(1);
+            $properties        = ArrayHelper::fromObject($table, false);
             $this->_cache[$pk] = ArrayHelper::toObject($properties);
 
             // Convert the params field to an array.

administrator/components/com_plugins/src/Model/PluginModel.php

@@ -174,7 +174,7 @@ public function getItem($pk = null)
             }
 
             // Convert to an object before adding other data.
-            $properties             = $table->getProperties(1);
+            $properties             = ArrayHelper::fromObject($table, false);
             $this->_cache[$cacheId] = ArrayHelper::toObject($properties);
 
             // Convert the params field to an array.

administrator/components/com_users/src/Model/GroupModel.php

@@ -277,7 +277,7 @@ public function delete(&$pks)
                 if ($allow) {
                     // Fire the before delete event.
                     $beforeDeleteEvent = new UserGroupBeforeDeleteEvent($this->event_before_delete, [
-                        'data'    => $table->getProperties(), // @TODO: Remove data argument in Joomla 6, see UserGroupBeforeDeleteEvent
+                        'data'    => ArrayHelper::fromObject($table, false), // @TODO: Remove data argument in Joomla 6, see UserGroupBeforeDeleteEvent
                         'context' => $context,
                         'subject' => $table,
                     ]);
@@ -297,7 +297,7 @@ public function delete(&$pks)
 
                     // Trigger the after delete event.
                     $dispatcher->dispatch($this->event_after_delete, new UserGroupAfterDeleteEvent($this->event_after_delete, [
-                        'data'           => $table->getProperties(), // @TODO: Remove data argument in Joomla 6, see UserGroupAfterDeleteEvent
+                        'data'           => ArrayHelper::fromObject($table, false), // @TODO: Remove data argument in Joomla 6, see UserGroupAfterDeleteEvent
                         'deletingResult' => true, // @TODO: Remove deletingResult argument in Joomla 6, see UserGroupAfterDeleteEvent
                         'errorMessage'   => $this->getError(), // @TODO: Remove errorMessage argument in Joomla 6, see UserGroupAfterDeleteEvent
                         'context'        => $context,

administrator/components/com_users/src/Model/UserModel.php

@@ -353,7 +353,7 @@ public function delete(&$pks)
                     $dispatcher->dispatch(
                         $this->event_before_delete,
                         new BeforeDeleteEvent($this->event_before_delete, [
-                            'subject' => $table->getProperties(),
+                            'subject' => ArrayHelper::fromObject($table, false),
                         ])
                     );
 
@@ -423,7 +423,7 @@ public function block(&$pks, $value = 1)
                 unset($pks[$i]);
                 Factory::getApplication()->enqueueMessage(Text::_('COM_USERS_USERS_ERROR_CANNOT_BLOCK_SELF'), 'error');
             } elseif ($table->load($pk)) {
-                $old   = $table->getProperties();
+                $old   = ArrayHelper::fromObject($table, false);
                 $allow = $user->authorise('core.edit.state', 'com_users');
 
                 // Don't allow non-super-admin to delete a super admin
@@ -457,7 +457,7 @@ public function block(&$pks, $value = 1)
                             new BeforeSaveEvent($this->event_before_save, [
                                 'subject' => $old,
                                 'isNew'   => false,
-                                'data'    => $table->getProperties(),
+                                'data'    => ArrayHelper::fromObject($table, false),
                             ])
                         )->getArgument('result', []);
 
@@ -479,7 +479,7 @@ public function block(&$pks, $value = 1)
 
                         // Trigger the after save event
                         $dispatcher->dispatch($this->event_after_save, new AfterSaveEvent($this->event_after_save, [
-                            'subject'      => $table->getProperties(),
+                            'subject'      => ArrayHelper::fromObject($table, false),
                             'isNew'        => false,
                             'savingResult' => true,
                             'errorMessage' => null,
@@ -584,7 +584,7 @@ public function activate(&$pks)
         // Activate and send the notification email
         foreach ($pks as $i => $pk) {
             if ($table->load($pk)) {
-                $prevUserData = $table->getProperties();
+                $prevUserData = ArrayHelper::fromObject($table, false);
                 $allow        = $user->authorise('core.edit.state', 'com_users');
 
                 // Don't allow non-super-admin to edit the active status of a super admin
@@ -616,7 +616,7 @@ public function activate(&$pks)
                             new BeforeSaveEvent($this->event_before_save, [
                                 'subject' => $prevUserData,
                                 'isNew'   => false,
-                                'data'    => $table->getProperties(),
+                                'data'    => ArrayHelper::fromObject($table, false),
                             ])
                         )->getArgument('result', []);
 
@@ -635,7 +635,7 @@ public function activate(&$pks)
 
                         // Fire the after save event
                         $dispatcher->dispatch($this->event_after_save, new AfterSaveEvent($this->event_after_save, [
-                            'subject'      => $table->getProperties(),
+                            'subject'      => ArrayHelper::fromObject($table, false),
                             'isNew'        => false,
                             'savingResult' => true,
                             'errorMessage' => null,

administrator/components/com_workflow/src/Model/StagesModel.php

@@ -17,6 +17,7 @@
 use Joomla\CMS\MVC\Model\ListModel;
 use Joomla\Database\ParameterType;
 use Joomla\Database\QueryInterface;
+use Joomla\Utilities\ArrayHelper;
 
 // phpcs:disable PSR1.Files.SideEffects
 \defined('_JEXEC') or die;
@@ -201,7 +202,7 @@ public function getWorkflow()
             $table->load($workflowId);
         }
 
-        return (object) $table->getProperties();
+        return (object) ArrayHelper::fromObject($table, false);
     }
 
     /**

administrator/components/com_workflow/src/Model/TransitionModel.php

@@ -17,6 +17,7 @@
 use Joomla\CMS\Plugin\PluginHelper;
 use Joomla\Registry\Registry;
 use Joomla\String\StringHelper;
+use Joomla\Utilities\ArrayHelper;
 
 // phpcs:disable PSR1.Files.SideEffects
 \defined('_JEXEC') or die;
@@ -298,7 +299,7 @@ public function getWorkflow()
 
         $workflow->load($workflow_id);
 
-        return (object) $workflow->getProperties();
+        return (object) ArrayHelper::fromObject($workflow, false);
     }
 
     /**

administrator/components/com_workflow/src/Model/TransitionsModel.php

@@ -16,6 +16,7 @@
 use Joomla\CMS\MVC\Model\ListModel;
 use Joomla\Database\ParameterType;
 use Joomla\Database\QueryInterface;
+use Joomla\Utilities\ArrayHelper;
 
 // phpcs:disable PSR1.Files.SideEffects
 \defined('_JEXEC') or die;
@@ -246,6 +247,6 @@ public function getWorkflow()
             $table->load($workflowId);
         }
 
-        return (object) $table->getProperties();
+        return (object) ArrayHelper::fromObject($table, false);
     }
 }

components/com_contact/src/Model/FormModel.php

@@ -105,7 +105,7 @@ public function getItem($itemId = null)
             return false;
         }
 
-        $properties = $table->getProperties();
+        $properties = ArrayHelper::fromObject($table, false);
         $value      = ArrayHelper::toObject($properties);
 
         // Convert field to Registry.

components/com_content/src/Model/FormModel.php

@@ -98,7 +98,7 @@ public function getItem($itemId = null)
             return false;
         }
 
-        $properties = $table->getProperties(1);
+        $properties = ArrayHelper::fromObject($table, false);
         $value      = ArrayHelper::toObject($properties);
 
         // Convert attrib field to Registry.

components/com_tags/src/Model/TagModel.php

@@ -294,7 +294,7 @@ public function getItem($pk = null)
                     }
 
                     // Convert the Table to a clean object.
-                    $properties   = $table->getProperties(1);
+                    $properties   = ArrayHelper::fromObject($table, false);
                     $this->item[] = ArrayHelper::toObject($properties);
                 } catch (\RuntimeException $e) {
                     $this->setError($e->getMessage());

libraries/src/Table/Table.php

@@ -26,6 +26,7 @@
 use Joomla\Event\DispatcherInterface;
 use Joomla\Filesystem\Path;
 use Joomla\String\StringHelper;
+use Joomla\Utilities\ArrayHelper;
 
 // phpcs:disable PSR1.Files.SideEffects
 \defined('_JEXEC') or die;
@@ -45,7 +46,6 @@ abstract class Table extends \stdClass implements TableInterface, DispatcherAwar
     use LegacyErrorHandlingTrait;
     use LegacyPropertyManagementTrait;
 
-
     /**
      * Include paths for searching for Table classes.
      *
@@ -728,7 +728,7 @@ public function bind($src, $ignore = [])
         }
 
         // Bind the source value, excluding the ignored fields.
-        foreach ($this->getProperties() as $k => $v) {
+        foreach (ArrayHelper::fromObject($this, false) as $k => $v) {
             // Only process fields not in the ignore array.
             if (!\in_array($k, $ignore)) {
                 if (\array_key_exists($k, $src)) {
@@ -816,7 +816,7 @@ public function load($keys = null, $reset = true)
         $query = $db->createQuery()
             ->select('*')
             ->from($db->quoteName($this->_tbl));
-        $fields = array_keys($this->getProperties());
+        $fields = array_keys(ArrayHelper::fromObject($this, false));
 
         foreach ($keys as $field => $value) {
             // Check that $field is in the table.

libraries/src/User/User.php

@@ -879,7 +879,7 @@ public function load($id)
         }
 
         // Assuming all is well at this point let's bind the data
-        foreach ($table->getProperties() as $key => $value) {
+        foreach (ArrayHelper::fromObject($table, false) as $key => $value) {
             $this->$key = $value;
         }