MBS-4486: Trim leading and trailing brackets from comments
MBS-3748: Use parentid to easily create children attributes
Add Approve to searchable My Votes
MBS-4700: Update doc search submit button with new inline style
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.
MBS-4733, merge release group secondary types.
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-4649, split up vimeo/youtube url cleanups so the correct type can be selected.
MBS-4733, don't loop over release group ids when merging secondary release group types.
Merge branch 'mbs-4733'

* mbs-4733:
  MBS-4733, don't loop over release group ids when merging secondary release group types.
  MBS-4733, merge release group secondary types.
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.
When deleting release groups, delete any secondary types they have
Patch old upgrade script to also remove editor_watch_release_group_type rows
Merge branch 'mbs-4760'

* mbs-4760:
  When deleting release groups, delete any secondary types they have
Add coverart statistics tab with all available statistics.
`
MBS-4765: Update DB_SCHEMA_SEQUENCE for recent schema update
MBS-4681, clean_submitted_artist_credits should assume an artist credit to be empty when it has no name and no artist credit.