Currently the land and structure type field come after status, flags
and owners. This is different than animal and plant assets whose type
field comes immediately after the asset name. Since the land and
structure type are required fields it seems more appropriate to have
them included higher in the form.
Submitting the group asset action form without a group yields the following error:
Message TypeError: array_column(): Argument #1 ($array) must be of type array, null given in array_column()
(line 195 of /var/www/html/web/profiles/farm/modules/asset/group/src/Form/AssetGroupActionForm.php)
One fix is to use the null coalescing operator to return an empty array when the form value is null.
We intentionally include the project in farm.info.yml, because we handle
packaging ourselves via GitHub Actions.
This line was added in https://www.drupal.org/project/farm/issues/3336698
This caused the following CodeSniffer warning:
FILE: /opt/drupal/web/profiles/farm/farm.info.yml
----------------------------------------------------------------------
FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
----------------------------------------------------------------------
1 | WARNING | Remove "project" from the info file, it will be added
| | by drupal.org packaging automatically
----------------------------------------------------------------------
This was originally added so that the Price quantity type could set its
default measure to "value" all the time. This was back when price
quantities were part of farmOS core. It was then moved to the
farm_ledger contrib module, and is being refactored such that it will no
longer need to make use of this default_measure configuration.
PR #603 changed the "Laboratory" field from a simple text field to a
taxonomy term reference field. But it did not update the v1 migration
logic accordingly.
See https://github.com/farmOS/farmOS/pull/603