Skip to content

[4.0] Move custom fields from submenu to main menu in contact#23366

Closed
bembelimen wants to merge 12 commits intojoomla:4.0-devfrom
bembelimen:Move-custom-fields-to-main-menu
Closed

[4.0] Move custom fields from submenu to main menu in contact#23366
bembelimen wants to merge 12 commits intojoomla:4.0-devfrom
bembelimen:Move-custom-fields-to-main-menu

Conversation

@bembelimen
Copy link
Contributor

Summary of Changes

At the moment the custom fields are in the submenu for contact. With this PR The two submenu items are moved to the main menu like in the other extensions.

Testing Instructions

  • Apply patch
  • Reinstall Joomla!
  • Check menu items for contact
  • disable/enable custom fields for contact to check display/hide

@chmst
Copy link
Contributor

chmst commented Dec 28, 2018

I have tested this item ✅ successfully on 1846d32


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

@Hackwar
Copy link
Member

Hackwar commented Dec 30, 2018

I have tested this item ✅ successfully on 1846d32

Tested successfully. We shouldd really merge in the testing sample data plugin and drop the SQL files.


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

@Quy
Copy link
Contributor

Quy commented Dec 30, 2018

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Dec 30, 2018
@infograf768 infograf768 changed the title Move custom fields from submenu to main menu in contact [4.0] Move custom fields from submenu to main menu in contact Dec 30, 2018
@brianteeman
Copy link
Contributor

can you fix the conflicts please

…om-fields-to-main-menu

# Conflicts:
#	administrator/modules/mod_menu/Menu/CssMenu.php
@ghost ghost added the J4 Issue label Apr 5, 2019
@ghost ghost removed the J4 Issue label Apr 13, 2019
bembelimen and others added 4 commits April 16, 2019 22:42
…om-fields-to-main-menu

# Conflicts:
#	installation/sql/mysql/joomla.sql
#	installation/sql/mysql/sample_testing.sql
#	installation/sql/postgresql/joomla.sql
#	installation/sql/postgresql/sample_testing.sql
@brianteeman
Copy link
Contributor

@bembelimen can you fix the conflicts please and then we can get this merged

@wilsonge
Copy link
Contributor

Removing rtc until conflicts are fixed. Mainly just to avoid me re-reading this PR every 2 weeks when I go through the RTC pile. Whenever they are fixed someone can mark this back as RTC


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

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jun 23, 2019
brianteeman added a commit to brianteeman/joomla-cms that referenced this pull request Jun 23, 2019
This is some additional corrections to be applied after joomla#23366 to complete the removaal of the sidebar for com_contacts
@brianteeman
Copy link
Contributor

@bembelimen please can you fix the conflicts

@brianteeman
Copy link
Contributor

I tried to test this with the view to resolving the conflicts and submitting it as a new pr but it is not a correct PR or the code required has changed since this was created

  1. disable/enable custom fields for contact to check display/hide
    This had no effect

  2. The separator between contacts and fields is not rendered correctly
    <li class="divider" role="presentation"><a class="no-dropdown" href="#"><span class="sidebar-item-title"></span></a></li>
    Should be
    <li class="divider" role="presentation"><span></span></li>
    Otherwise it becomes a clickable and focusable item

wilsonge pushed a commit that referenced this pull request Jul 15, 2019
This is some additional corrections to be applied after #23366 to complete the removaal of the sidebar for com_contacts
@ghost
Copy link

ghost commented Sep 2, 2019

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

@brianteeman
Copy link
Contributor

brianteeman commented Oct 8, 2019

Found the reason that
disable/enable custom fields for contact to check display/hide did not work

The value for the component_id in the sql was 33 - it should be the id of the com_fields component in the extensions table which is 30

@brianteeman
Copy link
Contributor

@bembelimen has obviously abandoned this important PR so if one of the sql gurus could create a new pr that would be awesome and then people will be able to create fields for com_contacts again

cc / @alikon @richard67 @Quy

@alikon
Copy link
Contributor

alikon commented Oct 10, 2019

please test #26540

@Quy Quy closed this Nov 8, 2019
@Quy Quy added RTC This Pull Request is Ready To Commit and removed Conflicting Files labels Nov 8, 2019
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Nov 8, 2019
@bembelimen bembelimen deleted the Move-custom-fields-to-main-menu branch April 4, 2020 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants