Yes, this would be more reasonable.

I had initially considered:

```
if ! defined(Class['apache::mod::prefork]){
  fail("apache::mod::php requires apache::mod::prefork and is not compatible with other mpm_workers")
}
```

I realised this duplicated a failed `require`, I just haven't moved it into the metaparameter.