PR-4.4-dev Pending

User tests: Successful: Unsuccessful:

avatar Kaushik1216
Kaushik1216
11 Jan 2023

Pull Request for Issue #39363

Summary of Changes
Replace isset() with array_key_exists() which also returns true when property has null as value.

Testing Instruction
Same as mention on issue https://github.com/joomla/joomla-cms/issues/39363

Result before appling this PR
On update column that have updated value null not change in database

Expected result after this PR
Column that have updated value null will also change in database

Link to documentations

No documentation changes for docs.joomla.org needed
No documentation changes for manual.joomla.org needed

avatar joomla-cms-bot joomla-cms-bot - change - 11 Jan 2023
Category Libraries
avatar Kaushik1216 Kaushik1216 - open - 11 Jan 2023
avatar Kaushik1216 Kaushik1216 - change - 11 Jan 2023
Status New Pending
avatar Quy Quy - change - 11 Jan 2023
The description was changed
avatar Quy Quy - edited - 11 Jan 2023
avatar Kaushik1216 Kaushik1216 - change - 11 Jan 2023
Labels Added: ?
avatar HLeithner
HLeithner - comment - 2 May 2023

This pull request has been automatically rebased to 4.3-dev.

avatar HLeithner
HLeithner - comment - 30 Sep 2023

This pull request has been automatically rebased to 4.4-dev.

avatar chmst chmst - test_item - 24 Feb 2024 - Tested successfully
avatar chmst
chmst - comment - 24 Feb 2024

I have tested this item ✅ successfully on 0db676b

Code looks good and works, but the original issue was not replicable on a joomla database table


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39607.

avatar Quy Quy - close - 5 Mar 2024
avatar Quy Quy - change - 5 Mar 2024
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2024-03-05 02:13:53
Closed_By Quy
avatar Quy Quy - change - 5 Mar 2024
Labels Removed: ?
avatar Quy Quy - change - 5 Mar 2024
Status Closed New
Closed_Date 2024-03-05 02:13:53
Closed_By Quy
Labels Added: PR-4.4-dev
avatar Quy Quy - change - 5 Mar 2024
Status New Pending
avatar Quy Quy - reopen - 5 Mar 2024
avatar HLeithner HLeithner - change - 24 Apr 2024
Title
Unable to store null value on update row #39363
[4.4] Unable to store null value on update row #39363
avatar HLeithner HLeithner - edited - 24 Apr 2024
avatar HLeithner
HLeithner - comment - 15 Nov 2024

This pull request has been automatically rebased to 5.2-dev.

avatar HLeithner HLeithner - change - 15 Nov 2024
Title
[4.4] Unable to store null value on update row #39363
[5.2] Unable to store null value on update row #39363
avatar HLeithner HLeithner - edited - 15 Nov 2024

Add a Comment

Login with GitHub to post a comment