Conflicting Files ? ? ? ? Failure

User tests: Successful: Unsuccessful:

avatar drmenzelit
drmenzelit
21 Mar 2019

Summary of Changes

Added several xml files for admin components to be able to create new menu items in admin menus

Testing Instructions

Expected result

Actual result

Documentation Changes Required

avatar drmenzelit drmenzelit - open - 21 Mar 2019
avatar drmenzelit drmenzelit - change - 21 Mar 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Mar 2019
Category Administration com_csp com_fields com_finder com_menus com_messages com_workflow Language & Strings
avatar wilsonge
wilsonge - comment - 21 Mar 2019

Do the fields views not need to have a component specified?

avatar drmenzelit
drmenzelit - comment - 21 Mar 2019

In my last commit I added a parameter on the files for fields to choose the correct component. The new file administrator/components/com_menus/Field/ComponentsfieldsField.php does the work (at least on my tests) :-)

avatar wilsonge wilsonge - change - 21 Mar 2019
Title
Add missing XML files for Admin Menu creation
[4.0] Add missing XML files for Admin Menu creation
avatar wilsonge wilsonge - change - 21 Mar 2019
Title
Add missing XML files for Admin Menu creation
[4.0] Add missing XML files for Admin Menu creation
avatar wilsonge wilsonge - edited - 21 Mar 2019
avatar chmst
chmst - comment - 21 Mar 2019

Selecting the component which use fields is not clear. There is no flag for "this component uses fields".
So the field selects components with param "custom_fields_enable":"1" .

After a fresh installation, only com_contact has set this parameter so other components, also com_content or com_users are not in the list.

We could change the install files, so that at least all core components have this parameter. But for 3rd party components?

avatar franz-wohlkoenig franz-wohlkoenig - change - 11 Apr 2019
Category Administration com_csp com_fields com_finder com_menus com_messages com_workflow Language & Strings Administration com_csp com_fields com_finder com_menus com_messages com_workflow
avatar brianteeman
brianteeman - comment - 16 Jul 2019

Can you look at resolving the conflicts and addressing the suggestions of @Quy so that this can be tested please.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 2 Sep 2019

Please resolve conflicting files so this PR can get tested at Worldwide Pizza, Bugs & Fun, October 19th

avatar drmenzelit drmenzelit - change - 2 Sep 2019
Labels Added: Conflicting Files ?
avatar joomla-cms-bot joomla-cms-bot - change - 2 Sep 2019
Category Administration com_csp com_fields com_finder com_menus com_messages com_workflow Administration com_csp com_fields com_finder com_menus com_messages com_workflow Language & Strings
avatar brianteeman
brianteeman - comment - 2 Sep 2019

@franz-wohlkoenig I really hope it can be fixed and merged before then

avatar drmenzelit
drmenzelit - comment - 2 Sep 2019

I have commited the suggested changes, I resolved the conflict on the language file and actualized my branch. I hope, it was correct and now the PR can be tested.

avatar VladikB VladikB - test_item - 3 Sep 2019 - Tested successfully
avatar VladikB
VladikB - comment - 3 Sep 2019

I have tested this item successfully on 408a5ca

New menu items:

  • Content Security Policy
    -> List All Content Security Policy Reports
  • Messaging
    -> List all Messages
  • Smart Search
    -> Smart Search Content Mapping
    -> Smart Search Filters
    -> Smart Search Indexed Content
    -> Smart Search results
    -> Smart Search Statistics
  • Workwlows
    -> List All Workflows
    This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24284.
avatar tecpromotion tecpromotion - test_item - 19 Oct 2019 - Tested successfully
avatar tecpromotion
tecpromotion - comment - 19 Oct 2019

I have tested this item successfully on 408a5ca


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

avatar Quy Quy - change - 19 Oct 2019
Status Pending Ready to Commit
avatar Quy
Quy - comment - 19 Oct 2019

RTC


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

avatar wilsonge wilsonge - change - 19 Oct 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-10-19 14:29:39
Closed_By wilsonge
Labels Added: ?
Removed: ?
avatar wilsonge wilsonge - close - 19 Oct 2019
avatar wilsonge wilsonge - merge - 19 Oct 2019
avatar wilsonge
wilsonge - comment - 19 Oct 2019

Thanks!

avatar wilsonge
wilsonge - comment - 19 Oct 2019

@chmst I think we should add that default value too the installation sql too

avatar wilsonge
wilsonge - comment - 19 Oct 2019

Codestyle fixed with 22feb63

Add a Comment

Login with GitHub to post a comment