? ? Pending

User tests: Successful: Unsuccessful:

avatar korenevskiy
korenevskiy
21 Jun 2021

Based on the problem of saving data with different types.
[4.0] Error when saving an object in JTable->save()

I've done a great job of implementing a typizations sub-framework for a Table class.
The problem is that before you could not create properties of the Table class with different types, you were forced to create properties without types. You had to use the default types.

But now in the modern world, you can use the Table class with fully typed properties. You don't need to worry that a DateTime property will cause an error. The DateTime property and other properties will be automatically converted to the database type before being saved. When the Table is loaded, all properties will be automatically converted to objects, such as a DateTime object.

Now, when programming, you will focus only on business logic.
.
At the same time, the updated Table class supports the previous usage scenarios. You don't need to redo the programs you've already created.
.
Bonus: The update made it possible to stop using the methods of the deprecated CMSObject class. The new Table class still inherits the CMSObject class, but now no method is used from this parent class.
.
The new Table class will allow you to discard the parent CMSObject class without loss in the future.
I checked this class, it does not violate the CMS in any way. But I still think that the code requires minor optimization.
.
I ask you not to ban my work, but to help solve the problem of complete typizations of classes, since technologies do not stand still.
In the modern world, typizations is a basic feature for frameworks.

avatar korenevskiy korenevskiy - open - 21 Jun 2021
avatar korenevskiy korenevskiy - change - 21 Jun 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Jun 2021
Category Unit Tests Repository Administration
avatar richard67
richard67 - comment - 21 Jun 2021

@korenevskiy You have made your PR against the staging branch so it shows more than 500 changed files and lots of conflicts. Please rebase to 4.0-dev or make a new PR.

avatar korenevskiy
korenevskiy - comment - 21 Jun 2021

@korenevskiy You have made your PR against the staging branch so it shows more than 500 changed files and lots of conflicts. Please rebase to 4.0-dev or make a new PR.

Thanks.

avatar korenevskiy korenevskiy - change - 21 Jun 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-06-21 13:15:39
Closed_By korenevskiy
Labels Added: ? ?
avatar korenevskiy korenevskiy - close - 21 Jun 2021
avatar korenevskiy korenevskiy - change - 21 Jun 2021
Status Closed New
Closed_Date 2021-06-21 13:15:39
Closed_By korenevskiy
avatar korenevskiy korenevskiy - change - 21 Jun 2021
Status New Pending
avatar korenevskiy korenevskiy - reopen - 21 Jun 2021
avatar korenevskiy korenevskiy - change - 21 Jun 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-06-21 13:16:19
Closed_By korenevskiy
avatar korenevskiy korenevskiy - close - 21 Jun 2021

Add a Comment

Login with GitHub to post a comment