Skip to content

Only delete properties set in data and keep ordering intact#15311

Merged
infograf768 merged 2 commits intojoomla:stagingfrom
rdeutz:fix_9129
Apr 15, 2017
Merged

Only delete properties set in data and keep ordering intact#15311
infograf768 merged 2 commits intojoomla:stagingfrom
rdeutz:fix_9129

Conversation

@rdeutz
Copy link
Copy Markdown
Contributor

@rdeutz rdeutz commented Apr 14, 2017

Pull Request for Issue #9129

Summary of Changes

Testing Instructions

  • Enable the user - profile plugin.
  • Within the plugin, under "User profile fields for registration and administrator user forms" select optional for the "Favorite Book" field.
  • under "User profile fields for profile edit form", select "disable" for the "Favorite Book" field.
  • Create a user in the administrator panel and fill in the "Favorite Book" field.
  • Go to the front-end, select "edit your profile" and hit submit.

Expected result

The "Favorite Book" field should stay the same.

Actual result

Before Patch: If you go to the administrator panel the "Favorite Book" field will be empty.
After Patch: If you go to the administrator panel the "Favorite Book" field will as before.

@AlexRed
Copy link
Copy Markdown
Contributor

AlexRed commented Apr 14, 2017

I have tested this item ✅ successfully on f47b163

Patch ok for me


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

wojsmol added a commit to wojsmol/joomla-cms that referenced this pull request Apr 14, 2017
@ghost
Copy link
Copy Markdown

ghost commented Apr 15, 2017

I have tested this item ✅ successfully on f47b163


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

@ghost
Copy link
Copy Markdown

ghost commented Apr 15, 2017

RTC after two successful tests.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Apr 15, 2017
@rdeutz rdeutz added this to the Joomla 3.7.0 milestone Apr 15, 2017
@infograf768 infograf768 merged commit 3a91455 into joomla:staging Apr 15, 2017
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Apr 15, 2017
@rdeutz rdeutz deleted the fix_9129 branch April 24, 2017 09:00
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.

4 participants