SearchOrders conditional filters

[redmorello]

Hi Matt,

With the SearchOrders specifications, is it possible to make some of them conditional. For example, on our orders listing page, we have a dropdown for the order Status. By default its set to "All" so we would use this search:

commerceOrders = _orderService.SearchOrders((where) => where
.FromStore(store.Id)
.And(where.IsFinalized())
.And(where.HasProperty(BlatchfordConstants.Commerce.Properties.Customer.CsiAccountId, csiAccount.CsiAccountId)),
(orderBy) => orderBy
.ByCreateDate(sortOrder),
pageNumber, pageSize);

But if the user selects "Cancelled", we would want to add an additional .And like this:

commerceOrders = _orderService.SearchOrders((where) => where
.FromStore(store.Id)
.And(where.IsFinalized())
.And(where.HasProperty(BlatchfordConstants.Commerce.Properties.Customer.CsiAccountId, csiAccount.CsiAccountId))
.And(where.HasOrderStatus(Guid.Parse(filterType))), // this line needs adding
(orderBy) => orderBy
.ByCreateDate(sortOrder),
pageNumber, pageSize);

we might need to add other optional search parameters i.e search by order line Sku etc

[mattbrailsford]

It should be possible within your function that you pass to the SearchOrders method to store the query and build it up dynamically. Here's a screenshot from the UC codebase where we do just that

[redmorello]

Thanks Matt, I've implemented that now and its working great.

Is it possible to search for sku on order lines with a "contains", similar to how the OrderNumber works?

if (!string.IsNullOrEmpty(searchTerm))
{
    var subQuery = where.HasOrderNumber(searchTerm, StringComparisonType.Contains).Or(where.HasOrderLineWithProduct(searchTerm));
    q = q.And(subQuery);
}

From looking at the interface IOrderQuerySpecificationFactory I can see that the HasOrderLineWithProduct uses:
x.ProductReference.Equals(ProductReferenceOrSku, StringComparison.InvariantCultureIgnoreCase)

[mattbrailsford]

Good question. It doesn't look like we can at the moment. I'm not sure why we didn't add that at the time, we must have just assumed that folks would supply the whole SKU to lookup.

Please raise this as an issue and I'll see what we can do to add the same StringComparisonType support.