Removed two redundant lines of code

`std::string type` is never used
what it seems to be purposed for is already being handled by the achievement_info class.

Since a reference (and therefore a pointer at the other end) to `type`
is created for the node->get(...) function, it seems likely to me that
`type` is never deallocated from memory and creates a tiny memory leak.