html work, code comments for tomorrow's tasks
This commit is contained in:
@@ -24,6 +24,9 @@ class SavedReport extends Model
|
||||
'options',
|
||||
];
|
||||
|
||||
//we will need a bit to catch and store the name of the report.
|
||||
//for now the blip above is creating the name, but can be confusing if multiple are made at once
|
||||
|
||||
public function checkmarkValue($property): string
|
||||
{
|
||||
// Assuming we're using the null object pattern,
|
||||
@@ -41,9 +44,10 @@ class SavedReport extends Model
|
||||
{
|
||||
// @todo: this method feels more like "radioShouldBeChecked" or something...
|
||||
|
||||
if (array_has($this->options, $property) && $this->options[$property] === $value) {
|
||||
return $return;
|
||||
}
|
||||
// if (array_has($this->options, $property) && $this->options[$property] === $value) {
|
||||
// return $return;
|
||||
// }
|
||||
// this is currently throwing an error. $property is coming through as a string and it needs to be an array
|
||||
|
||||
return null;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user