fix: createDatastorePredicate call was added

[frimfram]

Description of changes:

The function call to createDataStorePredicate is added to convert predicate in object format to DateStore predicate format. An example of generated code:

 const filterCriteriaObj = {
    and: [
      {
        field: 'age',
        operand: '10',
        operator: 'gt',
      },
      {
        field: 'lastName',
        operand: 'L',
        operator: 'beginsWith',
      },
    ],
  };

  const filterCriteria = createDataStorePredicate(filterCriteriaObj);

  const { buttonUser } = useDataStoreBinding({
    type: 'collection', // returns zero or one or many records
    model: User,
    criteria: filterCriteria,
    pagination: buttonUserPagination
  });

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[alharris-at]

You'll need to update snapshot tests as well.

[alharris-at]

Rather than creating the predicate declaration then compiled declaration statements, could you combine them into something like.

const buttonUserCondition = createDataStorePredicate({ <PREDICATE LITERAL> });

[frimfram]

Rather than creating the predicate declaration then compiled declaration statements, could you combine them into something like.

const buttonUserCondition = createDataStorePredicate({ <PREDICATE LITERAL> });

Not sure what you are talking about here: can you provide examples?

[dpilch]

He suggesting changing this:

 const filterCriteriaObj = {
    and: [
      {
        field: 'age',
        operand: '10',
        operator: 'gt',
      },
      {
        field: 'lastName',
        operand: 'L',
        operator: 'beginsWith',
      },
    ],
  };

  const filterCriteria = createDataStorePredicate(filterCriteriaObj);

To this:

  const filterCriteria = createDataStorePredicate({
    and: [
      {
        field: 'age',
        operand: '10',
        operator: 'gt',
      },
      {
        field: 'lastName',
        operand: 'L',
        operator: 'beginsWith',
      },
    ],
  });

[frimfram]

He suggesting changing this:

 const filterCriteriaObj = {
    and: [
      {
        field: 'age',
        operand: '10',
        operator: 'gt',
      },
      {
        field: 'lastName',
        operand: 'L',
        operator: 'beginsWith',
      },
    ],
  };

  const filterCriteria = createDataStorePredicate(filterCriteriaObj);

To this:

  const filterCriteria = createDataStorePredicate({
    and: [
      {
        field: 'age',
        operand: '10',
        operator: 'gt',
      },
      {
        field: 'lastName',
        operand: 'L',
        operator: 'beginsWith',
      },
    ],
  });

That is a good suggestion, but we should do what's agreed with Hector and also that version gives more flexibility with what's passed to the function param.