1. This hunk does not make sense as this query is now identical to the query in the `else` case. In this case, the if statement can be removed altogether.

2. It is unclear whether the expected query will be produced when `$albumYear` is `null`.
```
[00:21:00] <Raydiation> mjob: PR is up
[00:23:04] <bantu> Raydiation: errrrm
[00:23:15] <bantu> now the queries are identical
[00:23:41] <bantu> I guess it's supposed to be IS NULL if $albumYear is null
[00:23:53] <bantu> which would be the correct usage of "IS"
[00:24:03] <Raydiation> yeah, but this breaks on postgres atm
[00:24:21] <Raydiation> and dont think NULL will be used correctly
[00:24:39] <bantu> how does the query look now?
[00:24:49] <Raydiation> no idea
[00:25:05] <bantu> lol
[00:25:19] <bantu> so you have no idea whether the query returns the expected result?
[00:25:23] <Raydiation> all i know is that it doesnt break
[00:25:38] <Raydiation> and i can play music
```

3. There does not seem to be test coverage for `$albumYear = null`. See https://github.com/owncloud/music/blob/master/tests/php/unit/db/albumMapperTest.php#L128