diff --git a/app/Http/Controllers/OrderController.php b/app/Http/Controllers/OrderController.php index f5f12a1..8c982f0 100644 --- a/app/Http/Controllers/OrderController.php +++ b/app/Http/Controllers/OrderController.php @@ -108,6 +108,7 @@ class OrderController extends Controller $so->ordered_by = Auth::id(); $so->active = FALSE; $so->model = $order ? get_class($order) : NULL; + $so->recur_schedule = $po->price_recur_default; if ($order && $order->order_info) { $so->order_info = $order->order_info; diff --git a/app/Http/Controllers/ProductController.php b/app/Http/Controllers/ProductController.php index 09fec5e..0c8d212 100644 --- a/app/Http/Controllers/ProductController.php +++ b/app/Http/Controllers/ProductController.php @@ -28,6 +28,14 @@ class ProductController extends Controller ->sortBy('name') ->values(); + case 'App\Models\Product\Domain': + return Product\Domain::select(['id','supplier_item_id']) + ->with(['supplied.supplier_detail.supplier']) + ->get() + ->map(function($item) { return ['id'=>$item->id,'name'=>sprintf('%s: %s',$item->supplied->supplier_detail->supplier->name,$item->supplied->name)]; }) + ->sortBy('name') + ->values(); + case 'App\Models\Product\Email': return Product\Email::select(['id','supplier_item_id']) ->with(['supplied.supplier_detail.supplier']) diff --git a/app/Http/Controllers/ServiceController.php b/app/Http/Controllers/ServiceController.php index 267d1be..427cd8a 100644 --- a/app/Http/Controllers/ServiceController.php +++ b/app/Http/Controllers/ServiceController.php @@ -415,10 +415,12 @@ class ServiceController extends Controller $o->type->save(); - // @todo create a validator for service if ($request->post('invoice_next_at')) $o->invoice_next_at = $request->invoice_next_at; + if ($request->post('recur_schedule')) + $o->recur_schedule = $request->recur_schedule; + // Also update our service start_at date. // @todo We may want to make start_at/stop_at dynamic values calculated by the type records if ($request->post('start_at')) diff --git a/app/Http/Requests/ProductAddEdit.php b/app/Http/Requests/ProductAddEdit.php index ea49d93..71c6996 100644 --- a/app/Http/Requests/ProductAddEdit.php +++ b/app/Http/Requests/ProductAddEdit.php @@ -32,7 +32,7 @@ class ProductAddEdit extends FormRequest 'active' => 'sometimes|accepted', 'model' => 'sometimes|string', // @todo Check that it is a valid model type 'model_id' => 'sometimes|int', // @todo Check that it is a valid model type - 'accounting' => 'sometimes|string', + 'accounting' => 'nullable|string', ]; } } \ No newline at end of file diff --git a/app/Models/Product/Domain.php b/app/Models/Product/Domain.php index c66c86d..941ec68 100644 --- a/app/Models/Product/Domain.php +++ b/app/Models/Product/Domain.php @@ -12,6 +12,22 @@ final class Domain extends Type implements ProductItem { protected $table = 'product_domain'; + // Information required during the order process + protected array $order_attributes = [ + 'options.domain'=>[ + 'request'=>'options.domain', + 'key'=>'domain_name', + 'validation'=>'required|min:3', + 'validation_message'=>'Domain Name is a required field.', + ], + 'options.tld_id'=>[ + 'request'=>'options.tld_id', + 'key'=>'tld_id', + 'validation'=>'required|exists:tlds,id', + 'validation_message'=>'Domain TLD is a required field.', + ], + ]; + // The model that is referenced when this product is ordered protected string $order_model = ServiceDomain::class; diff --git a/app/Models/Service.php b/app/Models/Service.php index 184c461..58991be 100644 --- a/app/Models/Service.php +++ b/app/Models/Service.php @@ -800,7 +800,7 @@ class Service extends Model implements IDs */ public function getNameDetailAttribute() { - return ($this->type && $this->type->getServiceDescriptionAttribute()) ? $this->type->getServiceDescriptionAttribute() : 'No Description'; + return ($this->type && ($this->type->getServiceDescriptionAttribute() !== NULL)) ? $this->type->getServiceDescriptionAttribute() : 'No Description'; } /** diff --git a/app/Models/Service/Domain.php b/app/Models/Service/Domain.php index 7523408..83c57a8 100644 --- a/app/Models/Service/Domain.php +++ b/app/Models/Service/Domain.php @@ -28,11 +28,14 @@ class Domain extends Type return [ 'domain_name' => 'nullable|string|min:2', 'tld_id' => 'required|exists:tlds,id', - 'expire_at' => 'required|date', 'domain_registrar_id' => 'required|exists:domain_registrars,id', 'registrar_account' => 'nullable|string', 'registrar_ns' => 'nullable|string', 'registrar_username' => 'nullable|string', + 'connect_at' => 'nullable|date', + 'start_at' => 'nullable|date', + 'expire_at' => 'nullable|date|after:start_at', + 'recur_schedule' => 'nullable|int', // @todo insure value is at least 1 yr and not higher than Invoice::billing_periods ]; } diff --git a/database/migrations/2022_10_18_131249_domain_registrar_id_optional.php b/database/migrations/2022_10_18_131249_domain_registrar_id_optional.php new file mode 100644 index 0000000..c19ae92 --- /dev/null +++ b/database/migrations/2022_10_18_131249_domain_registrar_id_optional.php @@ -0,0 +1,28 @@ +Domain Name
Service Dates
+ +