Completion Suggester: Fix CompletionFieldMapper to correctly parse weight

Allows weight to be defined as a string representation of a positive integer

closes #8090
This seems to be the only "shared" portion for string/value.  Perhaps it could be moved out? Or just have a helper method, or even just leave the duplication (it is only 2 lines really, not bad).