Added logic for syncing fieldset info

Signed-off-by: snipe <snipe@snipe.net>
This commit is contained in:
snipe 2023-04-25 21:08:26 -07:00
parent 662b7787a1
commit 5eeca937cb

View file

@ -64,14 +64,16 @@ class CustomFieldsController extends Controller
* @return \Illuminate\Support\Facades\View * @return \Illuminate\Support\Facades\View
* @throws \Illuminate\Auth\Access\AuthorizationException * @throws \Illuminate\Auth\Access\AuthorizationException
*/ */
public function create() public function create(Request $request)
{ {
$this->authorize('create', CustomField::class); $this->authorize('create', CustomField::class);
$fieldsets = CustomFieldset::get();
return view('custom_fields.fields.edit', [ return view('custom_fields.fields.edit', [
'predefinedFormats' => Helper::predefined_formats(), 'predefinedFormats' => Helper::predefined_formats(),
'customFormat' => '', 'customFormat' => ''])
])->with('field', new CustomField()); ->with('fieldsets', $fieldsets)
->with('field', new CustomField());
} }
/** /**
@ -80,7 +82,7 @@ class CustomFieldsController extends Controller
* @see CustomFieldsController::createField() * @see CustomFieldsController::createField()
* @author [Brady Wetherington] [<uberbrady@gmail.com>] * @author [Brady Wetherington] [<uberbrady@gmail.com>]
* @since [v1.8] * @since [v1.8]
* @return Redirect * @return \Illuminate\Http\RedirectResponse
* @throws \Illuminate\Auth\Access\AuthorizationException * @throws \Illuminate\Auth\Access\AuthorizationException
*/ */
public function store(CustomFieldRequest $request) public function store(CustomFieldRequest $request)
@ -105,6 +107,7 @@ class CustomFieldsController extends Controller
"show_in_email" => $show_in_email, "show_in_email" => $show_in_email,
"is_unique" => $request->get("is_unique", 0), "is_unique" => $request->get("is_unique", 0),
"display_in_user_view" => $display_in_user_view, "display_in_user_view" => $display_in_user_view,
"auto_add_to_fieldsets" => $request->get("auto_add_to_fieldsets", 0),
"user_id" => Auth::id() "user_id" => Auth::id()
]); ]);
@ -116,10 +119,17 @@ class CustomFieldsController extends Controller
} }
if ($field->save()) { if ($field->save()) {
// Sync fields with fieldsets
if ($request->has('associate_fieldsets') && (count($request->get('associate_fieldsets')) > 0)) {
$field->fieldset()->sync(array_keys($request->get('associate_fieldsets')));
}
return redirect()->route('fields.index')->with('success', trans('admin/custom_fields/message.field.create.success')); return redirect()->route('fields.index')->with('success', trans('admin/custom_fields/message.field.create.success'));
} }
return redirect()->back()->withInput() return redirect()->back()->with('selected_fieldsets', $request->input('associate_fieldsets'))->withInput()
->with('error', trans('admin/custom_fields/message.field.create.error')); ->with('error', trans('admin/custom_fields/message.field.create.error'));
} }
@ -129,7 +139,7 @@ class CustomFieldsController extends Controller
* *
* @author [A. Gianotto] [<snipe@snipe.net>] * @author [A. Gianotto] [<snipe@snipe.net>]
* @since [v3.0] * @since [v3.0]
* @return Redirect * @return \Illuminate\Http\RedirectResponse
* @throws \Illuminate\Auth\Access\AuthorizationException * @throws \Illuminate\Auth\Access\AuthorizationException
*/ */
public function deleteFieldFromFieldset($field_id, $fieldset_id) public function deleteFieldFromFieldset($field_id, $fieldset_id)
@ -149,7 +159,6 @@ class CustomFieldsController extends Controller
} else { } else {
return redirect()->back()->withErrors(['message' => "Field is in use and cannot be deleted."]); return redirect()->back()->withErrors(['message' => "Field is in use and cannot be deleted."]);
} }
} }
return redirect()->back()->withErrors(['message' => "Error deleting field from fieldset"]); return redirect()->back()->withErrors(['message' => "Error deleting field from fieldset"]);
@ -162,7 +171,7 @@ class CustomFieldsController extends Controller
* *
* @author [Brady Wetherington] [<uberbrady@gmail.com>] * @author [Brady Wetherington] [<uberbrady@gmail.com>]
* @since [v1.8] * @since [v1.8]
* @return Redirect * @return \Illuminate\Http\RedirectResponse
* @throws \Illuminate\Auth\Access\AuthorizationException * @throws \Illuminate\Auth\Access\AuthorizationException
*/ */
public function destroy($field_id) public function destroy($field_id)
@ -191,12 +200,12 @@ class CustomFieldsController extends Controller
* @return \Illuminate\Support\Facades\View * @return \Illuminate\Support\Facades\View
* @throws \Illuminate\Auth\Access\AuthorizationException * @throws \Illuminate\Auth\Access\AuthorizationException
*/ */
public function edit($id) public function edit(Request $request, $id)
{ {
if ($field = CustomField::find($id)) { if ($field = CustomField::find($id)) {
$this->authorize('update', $field); $this->authorize('update', $field);
$fieldsets = CustomFieldset::get();
$customFormat = ''; $customFormat = '';
if ((stripos($field->format, 'regex') === 0) && ($field->format !== CustomField::PREDEFINED_FORMATS['MAC'])) { if ((stripos($field->format, 'regex') === 0) && ($field->format !== CustomField::PREDEFINED_FORMATS['MAC'])) {
$customFormat = $field->format; $customFormat = $field->format;
@ -205,6 +214,7 @@ class CustomFieldsController extends Controller
return view('custom_fields.fields.edit', [ return view('custom_fields.fields.edit', [
'field' => $field, 'field' => $field,
'customFormat' => $customFormat, 'customFormat' => $customFormat,
'fieldsets' => $fieldsets,
'predefinedFormats' => Helper::predefined_formats(), 'predefinedFormats' => Helper::predefined_formats(),
]); ]);
} }
@ -223,7 +233,7 @@ class CustomFieldsController extends Controller
* @author [A. Gianotto] [<snipe@snipe.net>] * @author [A. Gianotto] [<snipe@snipe.net>]
* @param int $id * @param int $id
* @since [v4.0] * @since [v4.0]
* @return Redirect * @return \Illuminate\Http\RedirectResponse
* @throws \Illuminate\Auth\Access\AuthorizationException * @throws \Illuminate\Auth\Access\AuthorizationException
*/ */
public function update(CustomFieldRequest $request, $id) public function update(CustomFieldRequest $request, $id)
@ -250,6 +260,7 @@ class CustomFieldsController extends Controller
$field->show_in_email = $show_in_email; $field->show_in_email = $show_in_email;
$field->is_unique = $request->get("is_unique", 0); $field->is_unique = $request->get("is_unique", 0);
$field->display_in_user_view = $display_in_user_view; $field->display_in_user_view = $display_in_user_view;
$field->auto_add_to_fieldsets = $request->get("auto_add_to_fieldsets", 0);
if ($request->get('format') == 'CUSTOM REGEX') { if ($request->get('format') == 'CUSTOM REGEX') {
$field->format = e($request->get('custom_format')); $field->format = e($request->get('custom_format'));
@ -262,6 +273,8 @@ class CustomFieldsController extends Controller
} }
if ($field->save()) { if ($field->save()) {
// Sync fields with fieldsets
$field->fieldset()->sync(array_keys($request->get('associate_fieldsets')));
return redirect()->route('fields.index')->with('success', trans('admin/custom_fields/message.field.update.success')); return redirect()->route('fields.index')->with('success', trans('admin/custom_fields/message.field.update.success'));
} }