MBS-3748: Use parentid to easily create children attributes
MBS-4700: Update doc search submit button with new inline style
Upgrade script: remove the 'Audiobook' primary type

This was a mistake and shouldn't exist. Instead, all audiobooks should use the
'Audiobook' secondary type and 'Other' as the primary type.
MBS-3748, set the default value for the parent dropdown on the FormHandler field, instead of doing that separately in FormRenderer.
MBS-4767: Fail edits that try and change the release group primary type to one that does not exist
Merge branch 'master' into mbs-3748-add-new-instruments

* master: (343 commits)
  Unify t/tests.t, t/edits.t and t/controllers.t
  Change --tests to take a regular expression
  Fix failing Relationship::LinkAttributeType::Index test
  Do not export COVER_ART_ARCHIVE_UPLOAD_PREFIXER from ShowDBDefs
  MBS-4765: Update DB_SCHEMA_SEQUENCE for recent schema update
  Patch old upgrade script to also remove editor_watch_release_group_type rows
  When deleting release groups, delete any secondary types they have
  MBS-4733, don't loop over release group ids when merging secondary release group types.
  MBS-4278, correctly refer to track data when creating recording edits.
  Use dedicated comment field type (MBS-4486)
  MBS-4621, do not call the display_artwork macro if we already know the release has no artwork.
  MBS-4520, correctly updated track positions in release editor after a track has been deleted.
  MBS-4680, correctly concatenate parts of cover art urls (without duplicate slashes).
  MBS-4733, merge release group secondary types.
  MBS-4687: Rewrite the Relationship::LinkType controller to use Role::Load and not have overlapping URLs
  Ensure that the CreateTable scripts use the same column order as
  Move specific relationship attributes to /relationship-attribute/{gid}
  MBS-4662: Move /relationship/attributes to /relationship-attributes
  Revert "Add <bdi> around a bunch of strings coming from the db (MBS-4568)"
  Skip HTML5 validation if the error is about bdi tags
  ...

Conflicts:
	root/relationship/linkattributetype/common.tt
MBS-3748: Update for compatability with 58e47f0ab

Also check UUID syntax to prevent sql errors.
MBS-3748: Go back to using form_row_select
MBS-3748: Fix bad merge conflict

tsk tsk, navap
MBS-4770: Add a test for /relationships/fake-fake, which should return status code 400.
Add a test for LinkAttributeType/Create
Merge branch 'mbs-4720'

* mbs-4720:
  Upgrade script: remove the 'Audiobook' primary type
Merge branch 'master' into mbs-3748-add-new-instruments

* master:
  Add a test for LinkAttributeType/Create
  MBS-4770: Add a test for /relationships/fake-fake, which should return status code 400.
  Upgrade script: remove the 'Audiobook' primary type
MBS-3748: Add test for creating linkattrs using a specified parentid
MBS-3748: Stop faking the test
Merge branch 'master' into mbs-1874-documentation-search

* master: (339 commits)
  Add a test for LinkAttributeType/Create
  MBS-4770: Add a test for /relationships/fake-fake, which should return status code 400.
  Unify t/tests.t, t/edits.t and t/controllers.t
  Change --tests to take a regular expression
  Fix failing Relationship::LinkAttributeType::Index test
  Do not export COVER_ART_ARCHIVE_UPLOAD_PREFIXER from ShowDBDefs
  MBS-4765: Update DB_SCHEMA_SEQUENCE for recent schema update
  Patch old upgrade script to also remove editor_watch_release_group_type rows
  When deleting release groups, delete any secondary types they have
  Upgrade script: remove the 'Audiobook' primary type
  MBS-4733, don't loop over release group ids when merging secondary release group types.
  MBS-4278, correctly refer to track data when creating recording edits.
  Use dedicated comment field type (MBS-4486)
  MBS-4621, do not call the display_artwork macro if we already know the release has no artwork.
  MBS-4520, correctly updated track positions in release editor after a track has been deleted.
  MBS-4680, correctly concatenate parts of cover art urls (without duplicate slashes).
  MBS-4733, merge release group secondary types.
  MBS-4687: Rewrite the Relationship::LinkType controller to use Role::Load and not have overlapping URLs
  Ensure that the CreateTable scripts use the same column order as
  Move specific relationship attributes to /relationship-attribute/{gid}
  ...
Merge remote branch 'navap/mbs-3748-add-new-instruments'

* navap/mbs-3748-add-new-instruments:
  MBS-3748: Stop faking the test
  MBS-3748: Add test for creating linkattrs using a specified parentid
  MBS-3748: Fix bad merge conflict
  MBS-3748: Go back to using form_row_select
  MBS-3748: Update for compatability with 58e47f0ab
  MBS-3748, set the default value for the parent dropdown on the FormHandler field, instead of doing that separately in FormRenderer.
  MBS-3748: Use parentid to easily create children attributes
Fix CAA indexing for indexing trigger by release label changes

Because the metadata.xml needs to be updated when a release's labels/catalog
numbers change, there is a trigger to kick off reindexing. However, the trigger
had an incorrect predicate where it tried to lookup releases by the
release_label id.

I've also added a join on cover art, so we only index stuff that actually has
cover art.
MBS-4772: Fix ModBot applying old edit work edits that change ISWCs
Group release groups on artist pages by distinct secondary type sets