Use std::iterator_traits to get the value_type of an EntityIterator

Instead of non-standard exported type 'Entity'.
Use a simpler way to define the value_type

The dune-grid test suite still passes...