? Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
11 Apr 2017

Pull Request for Issue #15230

Summary of Changes

delete the field if the uses which this.

Testing Instructions

Create a new field
Delete the field

Expected result

Cannot see the field in the database

Actual result

The field remains in the database

Documentation Changes Required

none

avatar zero-24 zero-24 - open - 11 Apr 2017
avatar zero-24 zero-24 - change - 11 Apr 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 Apr 2017
Category Administration com_fields
avatar uglyeoin
uglyeoin - comment - 11 Apr 2017

It didn't say it failed, neither did the original. But my DB stays the same.

avatar zero-24
zero-24 - comment - 11 Apr 2017

@uglyeoin this just do this for new "deletions" and not for the you allready did so you need to repeate the
creation and deletion ;)

avatar uglyeoin
uglyeoin - comment - 11 Apr 2017

@zero-24 I thought that is what I did but I will try again

avatar zero-24
zero-24 - comment - 11 Apr 2017

Great. Please make sure the changes in the code are applyed ?

avatar ot2sen
ot2sen - comment - 11 Apr 2017

Couldn´t reproduce issue - before - adding patch. The field was removed from database when emptying trash. This was for an article field. Does this happen elsewhere?

avatar alikon
alikon - comment - 12 Apr 2017

couldn't reproduce issue before patch, i've tested with user fields

avatar uglyeoin
uglyeoin - comment - 12 Apr 2017

@ot2sen argh, don't tell me there was no issue

avatar ot2sen
ot2sen - comment - 12 Apr 2017

@uglyeoin there might have been, I just couldn´t see it here and therefore unable to test a fix for something that didn´t happen :)
It was removed from db when emptying trash without any fixes.

avatar Quy
Quy - comment - 13 Apr 2017

Unable to reproduce the issue before the patch. Created and deleted fields/field groups under Users, Content, and Contacts successfully without orphans.

avatar zero-24
zero-24 - comment - 14 Apr 2017

Closing than. Thanks for testing.

avatar zero-24 zero-24 - change - 14 Apr 2017
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2017-04-14 11:57:29
Closed_By zero-24
Labels Added: ?
avatar zero-24 zero-24 - close - 14 Apr 2017
avatar uglyeoin
uglyeoin - comment - 15 Apr 2017

@zero-24 sorry, tired eyes my bad

Add a Comment

Login with GitHub to post a comment