Cannot remove value in repeatable

[cod3rshotout]

Hello, could someone help me on this issue please?

Laravel-Backpack/CRUD#5778

@karandatwani92

[backpack-ai-bot[bot]]

I understand you're having an issue with removing values in a repeatable field in Backpack. Based on the knowledge sources, there have been several issues related to repeatable fields in Backpack over time.

From what I can gather, there have been a few common issues with repeatable fields:

1. In older versions, when deleting all entries in a repeatable field, it would return a string "[]" instead of null, which was fixed in version 4.2 where empty repeatable fields now save as null in the database GitHub Issue #2820.

2. There was an issue with the default min_rows setting in repeatable fields, where even if you removed all entries, an empty one would appear again after saving. This was addressed in a PR that changed the default min_rows to 0 GitHub PR #4070.

3. There have been issues with uploaded files in repeatable fields not being properly deleted GitHub Issue #5411.

Without more specific details about your particular issue, it's difficult to provide a targeted solution. The GitHub issue you linked (#5778) isn't included in the knowledge sources provided.

I would recommend:

1. Make sure you're using the latest version of Backpack, as many repeatable field issues have been fixed in newer versions.
2. Check if your repeatable field has a min_rows setting that might be preventing complete removal.
3. If you're using file uploads within repeatable fields, there might be specific issues related to that functionality.

If you could provide more details about your specific issue with the repeatable field, I might be able to offer more targeted assistance.

ℹ️ Please note our AI Bot does not reply to follow-ups, this will be the only AI answer in this thread. If the AI helped you find the answer, please mark it as answered or write details below so it can learn. Then close the conversation. If not, please give more information for when a human reads this. ℹ️

Relevant sources:

Github Issue (closed): [Bug] Repeatable with field select2_multiple and relations
Github Issue (closed): [Bug] Repeatable field issue after update to Version 5
Github Issue (closed): [Bug] jQuery.Deferred exception: Cannot read property 'top' of undefined TypeError: Cannot read property 'top' of undefined
[4.2] make default min_rows in repeatable zero
Github Issue (closed): remove Button undefined variable name.
Github Issue (closed): [Bug] Illegal offset type when I am using repeatable to store pivot data
Handle undefined values for repeatable
Github Issue (closed): [Bug] Error in repeatable Field "Uncaught TypeError: Cannot read properties of undefined (reading 'subfieldsCallbacks')"
Github Issue (closed): Mutator not called with Repeatable field
Github Issue (closed): [Bug] Repeatable values not filled when open edit form (backpack pro)
Github Issue (closed): [Bug] It's impossible to remove the last role/permission from the user.
Github Issue (closed): [Bug] Select2 filter UI - previous selected value still visible after clearing
Github Issue (closed): [Bug] error on replacing repeatable subfield image when first saving it empty
Github Issue (closed): [Bug] Uploaded files cannot be deleted
Github Issue (closed): Repeatable field return '[]' string after deleting all entries

[cod3rshotout]

I managed to fix this using this code as trait:

 public function update()
 {

$dealershipId = $this->crud->getCurrentEntry()->id;

$selectedUsers = request()->input('users', []);

request()->request->remove('users');

$response = $this->traitUpdate();

DB::table('users')
  ->where('dealership_id', $dealershipId)
  ->update(['dealership_id' => null]);

if (!empty($selectedUsers)) {
  User::whereIn('id', $selectedUsers)->update(['dealership_id' => $dealershipId]);
}

return $response;
}

two things yet:

1. The update operation have a bug that must be fixed asap to handle this situation.
2. The inline operation does not appear.

[jcastroa87]

Hello @cod3rshotout I reply you original issue, because I need to fully replicate this to make sure there is an issue and not a bad approach.

Cheers.

[jcastroa87]

I will close this issue because is repeat with 5778

Cheers.