Fix for issue #6165 (#6168)

* Fix problem when using ValidatingTrait

* Checking that email alerts are enabled when trying to send expected check-in alerts (fix for issue #6169)
This commit is contained in:
Juho Taipale 2018-09-13 08:49:50 +03:00 committed by snipe
parent e45b4efa1a
commit 05b03df600
3 changed files with 25 additions and 13 deletions

View file

@ -57,17 +57,12 @@ class SendExpectedCheckinAlerts extends Command
}
}
// Send a rollup to the admin, if settings dictate
$recipient = new \App\Models\Recipients\AlertRecipient();
if (($assets) && ($assets->count() > 0) && ($settings->alert_email!='')) {
if (($assets) && ($assets->count() > 0) && ($settings->alerts_enabled && $settings->alert_email != '')) {
$recipient->notify(new ExpectedCheckinAdminNotification($assets));
}
}
}

View file

@ -12,9 +12,14 @@ use Illuminate\Validation\Rule;
class CustomField extends Model
{
use ValidatingTrait, UniqueUndeletedTrait;
public $guarded=["id"];
public static $PredefinedFormats=[
use ValidatingTrait,
UniqueUndeletedTrait;
public $guarded = [
"id"
];
public static $PredefinedFormats = [
"ANY" => "",
"CUSTOM REGEX" => "",
"ALPHA" => "alpha",
@ -31,6 +36,14 @@ class CustomField extends Model
"BOOLEAN" => "boolean",
];
/**
* Validation rules.
* At least empty array must be provided if using ValidatingTrait.
*
* @var array
*/
protected $rules = [];
/**
* The attributes that are mass assignable.
*
@ -57,7 +70,6 @@ class CustomField extends Model
*/
public static $table_name = "assets";
/**
* Convert the custom field's name property to a db-safe string.
*

View file

@ -7,10 +7,16 @@ use Watson\Validating\ValidatingTrait;
class CustomFieldset extends Model
{
use ValidatingTrait;
protected $guarded=["id"];
public $rules=[
"name" => "required|unique:custom_fieldsets"
/**
* Validation rules
* @var array
*/
protected $rules = [
"name" => "required|unique:custom_fieldsets"
];
/**
@ -21,7 +27,6 @@ class CustomFieldset extends Model
* @var boolean
*/
protected $injectUniqueIdentifier = true;
use ValidatingTrait;
/**