MBS-4486: Trim leading and trailing brackets from comments
Add Approve to searchable My Votes
MBS-4662: Move /relationship/attributes to /relationship-attributes

/relationship/attributes/{gid}/edit does not work, because Catalyst assumes that
'attributes' is a type of relationship type, like 'artist-artist'. I've fixed
this overlap by moving relationship attributes to live in the
/relationship-attributes namespace.

On reading this code I've taken the liberty to simplify it to use the 'Load'
role for loading, and also tested that editing works (preventing regressions)
and also that a few other links still work as I broke them while developing
this.
Move specific relationship attributes to /relationship-attribute/{gid}
MBS-4687: Rewrite the Relationship::LinkType controller to use Role::Load and not have overlapping URLs

Like the move for LinkAttributeTypes, LinkType has now been simplified to use
Role::Load. I've added a separate chained action for sypecific types of
relationships - such as artist-artist or label-work. This chained action handles
validation, to disallow people trying to create moose-pony relationships, for
example.

The form has been changed to not directly depend on values in $c->stash, that's
very ugly. Instead, the link type tree is now a required attribute on the form
and must be explicitly passed in.

Tests have been added to make sure all links work correctly, along with a test
for the broken 'create' link.
Merge branch 'master' into mbs-4486-debracket-comments

* master: (655 commits)
  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).
  Ensure that the CreateTable scripts use the same column order as
  Revert "Add <bdi> around a bunch of strings coming from the db (MBS-4568)"
  Skip HTML5 validation if the error is about bdi tags
  Require Test::WWW::Selenium::Catalyst => 0.07 in Makefile.PL and carton.lock.
  Add Net::CoverArtArchive to carton.lock
  Fix creating release groups from the release editor
  Update the "simple" views
  MBS-2532, make sure ipi_codes is specified when creating artists and labels in the release editor.
  Fix loading of historic create relationship edits
  Reset track_id_seq during upgrade
  Also fix add-release.html
  Attempt to fix selenium.t
  Fix invalid CHECK for ISWCs
  MBS-842, add a "reset track numbers" button.
  MBS-842, do not move free-text track numbers when the associated track is moved.
  Show primary and secondary release group types in the sidebar
  Extend release and release group type filtering to support secondary types
  ...
Use dedicated comment field type (MBS-4486)
MBS-4765: Update DB_SCHEMA_SEQUENCE for recent schema update
Do not export COVER_ART_ARCHIVE_UPLOAD_PREFIXER from ShowDBDefs

This is a function and should only be called, not exported.
Merge remote branch 'navap/mbs-4765-schema-sequence'

* navap/mbs-4765-schema-sequence:
  MBS-4765: Update DB_SCHEMA_SEQUENCE for recent schema update
Merge remote branch 'demosdemon/MBS-4697'

* demosdemon/MBS-4697:
  Add Approve to searchable My Votes
Merge branch 'mbs-4662'

* mbs-4662:
  MBS-4687: Rewrite the Relationship::LinkType controller to use Role::Load and not have overlapping URLs
  Move specific relationship attributes to /relationship-attribute/{gid}
  MBS-4662: Move /relationship/attributes to /relationship-attributes
Merge remote branch 'navap/mbs-4486-debracket-comments'

* navap/mbs-4486-debracket-comments:
  Use dedicated comment field type (MBS-4486)
  MBS-4486: Trim leading and trailing brackets from comments
Fix failing Relationship::LinkAttributeType::Index test

The links exist, but the test is checking for the wrong link.
Change --tests to take a regular expression

I'm often interesting in running all tests in a certain folder (especially for
controller tests), so taking a regular expresison is more flexible. It still
allows the previous behavior, but just expands upon it a bit.
Unify t/tests.t, t/edits.t and t/controllers.t

With the new commandline_override taking a regular expression, there's no need
to have these all separate.