Added show() redirect for CustomFields::show()

Signed-off-by: snipe <snipe@snipe.net>
This commit is contained in:
snipe 2021-04-20 15:09:23 -07:00
parent d40fe1b683
commit e286ff0be3

View file

@ -40,6 +40,24 @@ class CustomFieldsController extends Controller
} }
/**
* Just redirect the user back if they try to view the details of a field.
* We already show those details on the listing page.
*
* @see CustomFieldsController::storeField()
* @author [A. Gianotto] [<snipe@snipe.net>]
* @since [v5.1.5]
* @return Redirect
* @throws \Illuminate\Auth\Access\AuthorizationException
*/
public function show()
{
return redirect()->route("fields.index");
}
/** /**
* Returns a view with a form to create a new custom field. * Returns a view with a form to create a new custom field.
* *
@ -135,15 +153,15 @@ class CustomFieldsController extends Controller
{ {
$field = CustomField::find($field_id); $field = CustomField::find($field_id);
$this->authorize('delete', $field); $this->authorize('delete', $field);
if ($field->fieldset->count()>0) { if ($field->fieldset->count()>0) {
return redirect()->back()->withErrors(['message' => "Field is in-use"]); return redirect()->back()->withErrors(['message' => "Field is in-use"]);
}
$field->delete();
return redirect()->route("fields.index")
->with("success", trans('admin/custom_fields/message.field.delete.success'));
} }
$field->delete();
return redirect()->route("fields.index")
->with("success", trans('admin/custom_fields/message.field.delete.success'));
}
/** /**