Fix bug for dynamic number of subdomains

The gmsh reader was passing the wrong number of subdomains to the factory class and in turn, the traits were set incorrectly.

I introduced this bug myself !45 (merged), fortunately this was never released!