diff --git a/devdocs/specs/core-workorder.txt b/devdocs/specs/core-workorder.txt
index 8611279c..40caa0da 100644
--- a/devdocs/specs/core-workorder.txt
+++ b/devdocs/specs/core-workorder.txt
@@ -107,7 +107,7 @@ public class BuildingController : Controller {
If the client updates part of the wo graph, only that exact record really needs dependency checking.
There *is* however business rules that might take hold but that's all at the server and not related to concurrency directly
For example, on any change to the wo graph the server has to see if the wo is still editable and hasn't been locked or user's rights changed
- But that's not strictly concurrency related in teh sense that another user change the *same* record being updated
+ But that's not strictly concurrency related in the sense that another user change the *same* record being updated
So, for v8 as long as it can handle a portional update to part of the graph and uses the concurrency of that exact record to check then it sidesteps a lot of multi-user scenarios
This was only an issue in v7 due to it using only the wo header itself as the source of concurrency checking which would *always* involve the whole graph in any change anywhere
diff --git a/server/AyaNova/Controllers/WorkOrderController.cs b/server/AyaNova/Controllers/WorkOrderController.cs
index a20f720f..00ecbb4a 100644
--- a/server/AyaNova/Controllers/WorkOrderController.cs
+++ b/server/AyaNova/Controllers/WorkOrderController.cs
@@ -237,16 +237,16 @@ namespace AyaNova.Api.Controllers
//but maybe handy? Like do I need delete on entire woitems collection?
//WorkOrder/{woid}/WorkorderItems <- all workorderitems, post to add new, put to update all as a collection
//WorkOrder/{woid}/WorkOrderItems/{woitemid} <- CRUD single woitemid
-//https://docs.microsoft.com/en-us/azure/architecture/best-practices/api-design#define-operations-in-terms-of-http-methods
-/*
-Resource POST GET PUT DELETE
-/customers Create a new customer Retrieve all customers Bulk update of customers Remove all customers
-/customers/1 Error Retrieve the details for customer 1 Update the details of customer 1 if it exists Remove customer 1
-/customers/1/orders Create a new order for customer 1 Retrieve all orders for customer 1 Bulk update of orders for customer 1 Remove all orders for customer 1
-*/
+ //https://docs.microsoft.com/en-us/azure/architecture/best-practices/api-design#define-operations-in-terms-of-http-methods
+ /*
+ Resource POST GET PUT DELETE
+ /customers Create a new customer Retrieve all customers Bulk update of customers Remove all customers
+ /customers/1 Error Retrieve the details for customer 1 Update the details of customer 1 if it exists Remove customer 1
+ /customers/1/orders Create a new order for customer 1 Retrieve all orders for customer 1 Bulk update of orders for customer 1 Remove all orders for customer 1
+ */
-//So Post into a collection means create one item in that collection, never a whole collection being created at once
-//GET PUT and DELETE on a collecdtion always mean the entire collection, POST is the outlier here
+ //So Post into a collection means create one item in that collection, never a whole collection being created at once
+ //GET PUT and DELETE on a collecdtion always mean the entire collection, POST is the outlier here
// ///
@@ -363,19 +363,53 @@ Resource POST GET PUT DELETE
- //TODO: I can see adding or updating a collection of workorderitems but not deleting an entire collection
+
+ ///
+ /// Create WorkOrderItem
+ ///
+ ///
+ /// Automatically filled from route path, no need to specify in body
+ ///
+ [HttpPost("/items")]
+ public async Task PostWorkOrderItem([FromBody] WorkOrderItem newObject, ApiVersion apiVersion)
+ {
+ //NOTE: we don't need the workorder id in the route because the workorder item must contain the workorder id anyway
+
+
+ if (!serverState.IsOpen)
+ return StatusCode(503, new ApiErrorResponse(serverState.ApiErrorCode, null, serverState.Reason));
+
+ //Instantiate the business object handler
+ WorkOrderBiz biz = WorkOrderBiz.GetBiz(ct, HttpContext);
+
+ //If a user has change roles
+ if (!Authorized.HasCreateRole(HttpContext.Items, biz.BizType))
+ return StatusCode(403, new ApiNotAuthorizedResponse());
+
+ if (!ModelState.IsValid)
+ return BadRequest(new ApiErrorResponse(ModelState));
+
+ // //Create and validate
+ // WorkOrderItem o = await biz.CreateAsync(newObject);
+ // if (o == null)
+ // return BadRequest(new ApiErrorResponse(biz.Errors));
+ // else
+ // return CreatedAtAction(nameof(WorkOrderController.GetWorkOrder), new { id = o.Id, version = apiVersion.ToString() }, new ApiCreatedResponse(o));
+ return NoContent();
+ }
///
/// Delete WorkOrderItem
- ///
- ///
+ ///
///
/// Ok-no content
- [HttpDelete("{WorkOrderId}/items/{WorkOrderItemId}")]
- public async Task DeleteWorkOrderItem([FromRoute] long workOrderId, [FromRoute] long workOrderItemId)
+ [HttpDelete("/items/{WorkOrderItemId}")]
+ public async Task DeleteWorkOrderItem([FromRoute] long workOrderItemId)
{
+ //NOTE: we don't need the workorder id in the route because the workorder item must contain the workorder id anyway
+
//WorkOrder/{woid}/WorkorderItems <- all workorderitems, post to add new, put to update all as a collection
//WorkOrder/{WorkOrderId}/WorkorderItems
if (!serverState.IsOpen)
@@ -390,12 +424,17 @@ Resource POST GET PUT DELETE
if (!Authorized.HasDeleteRole(HttpContext.Items, biz.BizType))
return StatusCode(403, new ApiNotAuthorizedResponse());
- var o = await biz.GetAsync(workOrderId, false);
- if (o == null)
- return NotFound(new ApiErrorResponse(ApiErrorCode.NOT_FOUND));
- //Make sure the item exists first before getting into it
- if (!o.WorkorderItems.Exists(m => m.Id == workOrderItemId))
- return NotFound(new ApiErrorResponse(ApiErrorCode.NOT_FOUND));
+ //*******************************************************************************
+ //NOTE: I'm thinking there should be no db access in controller
+ //let the biz object return not found if necessary
+ //*******************************************************************************
+
+ // var o = await biz.GetAsync(workOrderId, false);
+ // if (o == null)
+ // return NotFound(new ApiErrorResponse(ApiErrorCode.NOT_FOUND));
+ // //Make sure the item exists first before getting into it
+ // if (!o.WorkorderItems.Exists(m => m.Id == workOrderItemId))
+ // return NotFound(new ApiErrorResponse(ApiErrorCode.NOT_FOUND));
@@ -416,13 +455,11 @@ Resource POST GET PUT DELETE
///
/// Delete WorkOrderItemLabor
- ///
- ///
- ///
+ ///
///
/// Ok-no content
- [HttpDelete("{WorkOrderId}/items/{WorkOrderItemId}/labors/{WorkOrderItemLaborId}")]
- public async Task DeleteWorkOrderItemLabor([FromRoute] long workOrderId, [FromRoute] long workOrderItemId, [FromRoute] long workOrderItemLaborId)
+ [HttpDelete("/items/labors/{WorkOrderItemLaborId}")]
+ public async Task DeleteWorkOrderItemLabor([FromRoute] long workOrderItemLaborId)
{
if (!serverState.IsOpen)
return StatusCode(503, new ApiErrorResponse(serverState.ApiErrorCode, null, serverState.Reason));
@@ -436,19 +473,25 @@ Resource POST GET PUT DELETE
if (!Authorized.HasDeleteRole(HttpContext.Items, biz.BizType))
return StatusCode(403, new ApiNotAuthorizedResponse());
- var o = await biz.GetAsync(workOrderId, false);
- if (o == null)
- return NotFound(new ApiErrorResponse(ApiErrorCode.NOT_FOUND));
+ //*******************************************************************************
+ //NOTE: I'm thinking there should be no db access in controller
+ //let the biz object return not found if necessary
+ //*******************************************************************************
- //Get WorkorderItem
- var woitem = o.WorkorderItems.FirstOrDefault(m => m.Id == workOrderItemId);
- if (woitem == null)
- return NotFound(new ApiErrorResponse(ApiErrorCode.NOT_FOUND));
- //Get WorkOrderItemLabor
- var woitemlabor = woitem.WorkorderItemLabors.FirstOrDefault(m => m.Id == workOrderItemLaborId);
- if (woitem == null)
- return NotFound(new ApiErrorResponse(ApiErrorCode.NOT_FOUND));
+ // var o = await biz.GetAsync(workOrderId, false);
+ // if (o == null)
+ // return NotFound(new ApiErrorResponse(ApiErrorCode.NOT_FOUND));
+
+ // //Get WorkorderItem
+ // var woitem = o.WorkorderItems.FirstOrDefault(m => m.Id == workOrderItemId);
+ // if (woitem == null)
+ // return NotFound(new ApiErrorResponse(ApiErrorCode.NOT_FOUND));
+
+ // //Get WorkOrderItemLabor
+ // var woitemlabor = woitem.WorkorderItemLabors.FirstOrDefault(m => m.Id == workOrderItemLaborId);
+ // if (woitem == null)
+ // return NotFound(new ApiErrorResponse(ApiErrorCode.NOT_FOUND));
@@ -466,13 +509,11 @@ Resource POST GET PUT DELETE
///
/// Delete WorkOrderItemPart
- ///
- ///
- ///
+ ///
///
/// Ok-no content
- [HttpDelete("{WorkOrderId}/items/{WorkOrderItemId}/parts/{WorkOrderItemPartId}")]
- public async Task DeleteWorkOrderItemPart([FromRoute] long workOrderId, [FromRoute] long workOrderItemId, [FromRoute] long workOrderItemPartId)
+ [HttpDelete("/items/parts/{WorkOrderItemPartId}")]
+ public async Task DeleteWorkOrderItemPart([FromRoute] long workOrderItemPartId)
{
if (!serverState.IsOpen)
return StatusCode(503, new ApiErrorResponse(serverState.ApiErrorCode, null, serverState.Reason));
@@ -486,12 +527,18 @@ Resource POST GET PUT DELETE
if (!Authorized.HasDeleteRole(HttpContext.Items, biz.BizType))
return StatusCode(403, new ApiNotAuthorizedResponse());
- var o = await biz.GetAsync(workOrderId, false);
- if (o == null)
- return NotFound(new ApiErrorResponse(ApiErrorCode.NOT_FOUND));
- //Make sure the item exists first before getting into it
- if (!o.WorkorderItems.Exists(m => m.Id == workOrderItemId))
- return NotFound(new ApiErrorResponse(ApiErrorCode.NOT_FOUND));
+
+ //*******************************************************************************
+ //NOTE: I'm thinking there should be no db access in controller
+ //let the biz object return not found if necessary
+ //*******************************************************************************
+
+ // var o = await biz.GetAsync(workOrderId, false);
+ // if (o == null)
+ // return NotFound(new ApiErrorResponse(ApiErrorCode.NOT_FOUND));
+ // //Make sure the item exists first before getting into it
+ // if (!o.WorkorderItems.Exists(m => m.Id == workOrderItemId))
+ // return NotFound(new ApiErrorResponse(ApiErrorCode.NOT_FOUND));
diff --git a/server/AyaNova/biz/AyaType.cs b/server/AyaNova/biz/AyaType.cs
index 506a1937..170a4ee9 100644
--- a/server/AyaNova/biz/AyaType.cs
+++ b/server/AyaNova/biz/AyaType.cs
@@ -96,7 +96,7 @@ namespace AyaNova.Biz
//AyaNova.Biz.BizObjectNameFetcherDIRECT
//and in the CLIENT in ayatype.js
- //and need TRANSLATION KEYS because any type could show in the event log at teh client end
+ //and need TRANSLATION KEYS because any type could show in the event log at the client end
}
diff --git a/server/AyaNova/biz/BizRoles.cs b/server/AyaNova/biz/BizRoles.cs
index a55520cf..9f7b2d93 100644
--- a/server/AyaNova/biz/BizRoles.cs
+++ b/server/AyaNova/biz/BizRoles.cs
@@ -366,7 +366,7 @@ namespace AyaNova.Biz
{
Change = AuthorizationRoles.BizAdminFull | AuthorizationRoles.OpsAdminFull,
//Anyone can read it because they need to to open a form, but also in UI
- //only the bizadminlimited actually gets a link to see teh customization page
+ //only the bizadminlimited actually gets a link to see the customization page
ReadFullRecord = AuthorizationRoles.All
});
@@ -390,7 +390,7 @@ namespace AyaNova.Biz
//Only BizAdminFull can modify forms
Change = AuthorizationRoles.BizAdminFull,
//Anyone can read it because they need to to open a form, but also in UI
- //only the bizadminlimited actually gets a link to see teh customization page
+ //only the bizadminlimited actually gets a link to see the customization page
ReadFullRecord = AuthorizationRoles.All
});