? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
15 Jun 2019

Pull Request for Issue #25211.

Summary of Changes

Get request group fields recursively.

Testing Instructions

Create/modify view layout XML file to contain a nested field group but no immediate fields in request field group, e.g.:

<fields name="request">
	<fieldset name="request">
		<fields name="filter">
			<field
				name="id"
				type="category"
				label="JGLOBAL_FIELD_CATEGORIES_CHOOSE_CATEGORY_LABEL"
				description="JGLOBAL_FIELD_CATEGORIES_CHOOSE_CATEGORY_DESC"
				extension="com_content"
				show_root="true"
				required="true"
			/>
		</fields>
	</fieldset>
</fields>

Create a menu item for the view.
Inspect Link field value.

Expected result

Nested group field parameters included, e.g.:

index.php?option=com_content&view=categories&filter[id]=0

Actual result

Nested group field parameters not included, e.g.:

index.php?option=com_content&view=categories

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 15 Jun 2019
avatar SharkyKZ SharkyKZ - change - 15 Jun 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Jun 2019
Category Administration com_menus
avatar SharkyKZ SharkyKZ - change - 15 Jun 2019
The description was changed
avatar SharkyKZ SharkyKZ - edited - 15 Jun 2019
avatar SharkyKZ SharkyKZ - change - 15 Jun 2019
The description was changed
avatar SharkyKZ SharkyKZ - edited - 15 Jun 2019
avatar muesliflyer
muesliflyer - comment - 15 Jun 2019

This works for me. Thanks for the quick fix.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 16 Jun 2019
avatar SharkyKZ SharkyKZ - change - 16 Jun 2019
Labels Added: ?
avatar muesliflyer
muesliflyer - comment - 16 Jun 2019

I have tested this item successfully on 914b2c8


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

avatar muesliflyer muesliflyer - test_item - 16 Jun 2019 - Tested successfully
avatar muesliflyer
muesliflyer - comment - 16 Jun 2019

For the test, I installed Patch Tester 2.0.1 (the stable version). When fetching the patch data, I got an error:

Notice: Undefined offset: 0 in /var/www/bugtesting/libraries/joomla/database/query.php on line 351 {"success":false,"message":"Error inserting pull request data into the database: Column count doesn't match value count at row 1","messages":null,"data":null}

Then I installed the latest version 3.0.0 beta 4. Then it worked.

avatar infograf768
infograf768 - comment - 17 Jun 2019

Then I installed the latest version 3.0.0 beta 4. Then it worked.

??

avatar SharkyKZ
SharkyKZ - comment - 17 Jun 2019

@infograf768 it's about Patchtester.

avatar viocassel viocassel - test_item - 25 Jun 2019 - Tested successfully
avatar viocassel
viocassel - comment - 25 Jun 2019

I have tested this item successfully on 5bba89d


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 25 Jun 2019
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 25 Jun 2019

Status "Ready To Commit".

avatar HLeithner HLeithner - change - 25 Jun 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-06-25 12:38:50
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - close - 25 Jun 2019
avatar HLeithner HLeithner - merge - 25 Jun 2019
avatar HLeithner
HLeithner - comment - 25 Jun 2019

thx

Add a Comment

Login with GitHub to post a comment