More readable multi-line ruby in erbs

A couple of erb files had unnecessary start and end erb tags around every line in a multi-line block of ruby. More readable this way.

Add newline to align with blocks_by

`blocks_on.html.erb` is almost identical to `block_by.html.erb`. Add a newline to align with the other file (makes it easier to see the more subtle important difference)

Fix minor HTML tag mismatches

Fix some HTML tag mismatches and other glitches. These are minor, and don't appear to cause browsers any problems, but for the sake of correctness:
* A closing td which should be a closing div.
* A closing h3 for a corresponding h6
* Two tables with `<thead>` tag and missing `<tr>` around the `<th>` headings.

test change for thead fix

Fix some rubocop Naming/PredicateName warnings

Merge remote-tracking branch 'upstream/pull/3485'

Merge branch 'pull/3472'

Fix redirecting links to osmfoundation.org

Fix some links to osmfoundation.org pages which are redirecting (not broken links but this is just avoiding a slightly messy redirection)

Merge remote-tracking branch 'upstream/pull/3484'

Merge remote-tracking branch 'upstream/pull/3483'

Bump image_processing from 1.12.1 to 1.12.2

Bumps [image_processing](https://github.com/janko/image_processing) from 1.12.1 to 1.12.2.
- [Release notes](https://github.com/janko/image_processing/releases)
- [Changelog](https://github.com/janko/image_processing/blob/master/CHANGELOG.md)
- [Commits](https://github.com/janko/image_processing/compare/v1.12.1...v1.12.2)

---
updated-dependencies:
- dependency-name: image_processing
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Bump actions/checkout from 2.4.0 to 3

Bumps [actions/checkout](https://github.com/actions/checkout) from 2.4.0 to 3.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/checkout/compare/v2.4.0...v3)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

Fix rubocop Rails/TimeZone warnings

Enable open redirect protection

Merge remote-tracking branch 'upstream/pull/3482'

Update bundle

Remove redundant presence validation on belongs_to

There's no need for us to have it when rails does this for us.

Restore asset debugging in development mode

Fix opensearch icon & update attribution

Fixes #3478
Closes #3479

Localisation updates from https://translatewiki.net.

Add missing PG_MODULE_MAGIC to database function source

Merge remote-tracking branch 'upstream/pull/3474'

Bump eslint from 8.9.0 to 8.10.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.9.0 to 8.10.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v8.9.0...v8.10.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Localisation updates from https://translatewiki.net.

Merge remote-tracking branch 'upstream/pull/3470'

Enable active_record.belongs_to_required_by_default

This switches the logic so that belongs_to parent objects must exist
by default, and marks the optional ones explicitly. This is reflected
in the null/not_null status on the relevant db columns.

Merge remote-tracking branch 'upstream/pull/3469'

Add inverse_of to relationships that can't detect it automatically

Remove default values from id columns

In both the case of primary keys, and also foreign key references,
there's no need to set a default value.

This doesn't have a big impact in routine situations, but can be
very confusing when debugging corner cases.

Specify inverse_of for trace tags and points

Rails can't automatically detect bi-directional relationships when
`:foreign_key` is present. Adding the `inverse_of` helps when validating
traces and tags that haven't yet been saved.

Disable partial inserts

Switch to rails 7 default headers

These are basically all the same as what secure_headers
was already setting for us anyway.

Merge remote-tracking branch 'upstream/pull/3462'

Make sure the object layer is kept in front of the data layer

Don't highlight selected items in the data layer

Routing to the feature page is going to cause them to be overlaid
in orange anyway so there's no need for the blue highlight.

Update bundle

Deep copy the original style when highlighting an object

Fixes #3465

Enable automatic scope inversing

Use a button element for button_to

Apply bootstrap styling to "make public" button

Default to not specifying media restrictions on stylesheet tags

Enable verification of foreign keys in tests

Localisation updates from https://translatewiki.net.

Replace deprecated String.prototype.substr()

String.prototype.substr() is deprecated (see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/substr) so we replace it with slice() which works similarily but isn't deprecated.
Signed-off-by: Tobias Speicher <rootcommander@gmail.com>

Wrap test execution for better isolation

Enable some rails 7.x defaults

Localisation updates from https://translatewiki.net.

Default to allowing TLS for SMTP but without peer verification

Disable peer host name validation when sending email

Merge remote-tracking branch 'upstream/pull/3398'

Merge remote-tracking branch 'upstream/pull/3461'

Merge pull request #3414 from tomhughes/rails7

Update to rails 7.x

Merge pull request #3440 from mmd-osm/relationmemberlimit

Introduce relation member limit

Update to rails 7.0.2.2

Update documentation for how to confirm a new user account

Fixes #3460

Update bundle

Merge remote-tracking branch 'upstream/pull/3458'

Bump eslint from 8.8.0 to 8.9.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.8.0 to 8.9.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v8.8.0...v8.9.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Localisation updates from https://translatewiki.net.

Allow trace image URL to be configured in the CSP policy

Merge remote-tracking branch 'upstream/pull/3345'

Merge remote-tracking branch 'upstream/pull/3455'

Remove section on removed rake doc:app

Remove the section of contributing docs about how `rake doc:app` can be used to generate some sort of documentation. That will not work any more, since this task was removed from Rails v5 because people don't generally use it! https://stackoverflow.com/a/36804474/338265

Fix contrib doc to remove reference to travis

Remove the reference to Travis CI. Our CI pipeline now runs withing github actions.

Merge remote-tracking branch 'upstream/pull/3454'

Merge remote-tracking branch 'upstream/pull/3453'

Merge remote-tracking branch 'upstream/pull/3452'

Adds advancedcomp to the list of macOS Homebrew dependencies

Fix vagrant storage.yml config

Tell vagrant to copy the example storage.yml config file into place as per INSTALL.md instructions. Allows the migrations to run.

Updates macOS geckodriver installation command

Update to rails 6.1.4.6

Update bundle

Localisation updates from https://translatewiki.net.

Allow users to delete their own accounts

This PR allows users to delete their own accounts. The logic implemented matches
that currently used by the admins when they manually close accounts, although
there is room to be more complex in future e.g. completely removing accounts
with no content.

The error handling has been slightly adapted for namespaced controllers, by
anchoring the controller name with a leading forward slash.

Update bundle

Localisation updates from https://translatewiki.net.

Merge remote-tracking branch 'upstream/pull/3419'

Merge remote-tracking branch 'upstream/pull/3446'

Merge remote-tracking branch 'upstream/pull/3445'

Update to iD v2.20.4

Localisation updates from https://translatewiki.net.

Ensure that deactivate isn't available in production

It's only used as a workaround for factories not being able to create
pending users while keeping active as the default

Add extra user transitions needed by the administrators

Added some key-value pairs under `railway=`

Introduce relation member limit

Adds a new parameter `max_number_of_relation_members` in settings.yml

Update bundle

Index note comments by author and date

Fixes #3443

Merge remote-tracking branch 'upstream/pull/3442'

Merge remote-tracking branch 'upstream/pull/3439'

Update to iD v2.20.3

Localisation updates from https://translatewiki.net.

Bump eslint from 8.7.0 to 8.8.0

Bumps [eslint](https://github.com/eslint/eslint) from 8.7.0 to 8.8.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md)
- [Commits](https://github.com/eslint/eslint/compare/v8.7.0...v8.8.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

Fix route for "go public" button on the account edit page

Localisation updates from https://translatewiki.net.

Update bundle

Localisation updates from https://translatewiki.net.

Localisation updates from https://translatewiki.net.

Merge remote-tracking branch 'upstream/pull/3426'

Allow blank issue templates

This was originally intended in #3397 and I'm not sure why I set this
to false.

Merge remote-tracking branch 'upstream/pull/3420'

Update bundle