Greg Roach

I think throwing an exception is the correct action. The [container specification](https://www.php-fig.org/psr/psr-11/) lists specific exceptions that should be thrown. > return null instead? Containers can contain any type of object...

The code-check is complaining that you have `elseif` at the start of a line. It should be on the previous line, after the `}`. Would this be a simpler approach?...

> How to handle: > Added line #L153 was not covered by tests > and the others "Added line" You can ignore this. It says that there are no tests...

Thanks!

> an old MySQL database What version of MySQL?

The `HomePage` route doesn't actually display a page. It redirects to the default tree, with `TreePage` for visitors and `MyPage` for logged-in users. Since the user is currently logged in,...

@miqrogroove - you gave a :+1: but surely this won't help you. You've replaced `HomePage` - presumably with some sort of content page - and so replacing the current link...

We could only pre-fill it with the country name (e.g. "England" for the Censuses of England). But this would be the place recorded in the census itself, which wouldn't say...

We have two types of placenames, related to evidence and conclusion. The historic placenames, as recorded in sources, are part of the evidence. The contemporary placenames, as recorded in the...

This PR supercedes #215