bug ? PR-4.4-dev Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
17 Sep 2023

Pull Request for Issue #38642 .

Summary of Changes

redo of ex-#38655

Testing Instructions

have a list with multiple yes and or a radio articol field
set some articles

GET {{base_path}}/api/index.php/v1/content/articles

Actual result BEFORE applying this Pull Request

error 500

Expected result AFTER applying this Pull Request

no more error 500

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
5.00

avatar joomla-cms-bot joomla-cms-bot - change - 17 Sep 2023
Category Front End Plugins
avatar alikon alikon - open - 17 Sep 2023
avatar alikon alikon - change - 17 Sep 2023
Status New Pending
avatar antonbag antonbag - test_item - 18 Sep 2023 - Tested successfully
avatar antonbag
antonbag - comment - 18 Sep 2023

I have tested this item ✅ successfully on 3819ddf

Now It works :)


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

avatar MacJoom
MacJoom - comment - 18 Sep 2023

@alikon - Thanks for the PR. What is the test setup for radio buttons - i can see the error 500 with the multiple items in list type and the bug is fixed - but i cannot force the error with the radio buttons

avatar alikon
alikon - comment - 19 Sep 2023

create a radio field make it not required and not fill it in an article

without pr

you get
"serie-a": { "": null },

with the pr

"serie-a": [],

avatar MacJoom MacJoom - change - 19 Sep 2023
Labels Added: bug ? PR-4.4-dev
avatar Quy Quy - test_item - 19 Sep 2023 - Tested successfully
avatar Quy
Quy - comment - 19 Sep 2023

I have tested this item ✅ successfully on e012e72


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

avatar MacJoom MacJoom - change - 20 Sep 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-09-20 10:58:16
Closed_By MacJoom
avatar MacJoom MacJoom - close - 20 Sep 2023
avatar MacJoom MacJoom - merge - 20 Sep 2023
avatar MacJoom
MacJoom - comment - 20 Sep 2023

Thank you!

Add a Comment

Login with GitHub to post a comment