-
Type: Improvement
-
Resolution: Done
-
Priority: Major - P3
-
Affects Version/s: None
-
Component/s: None
-
None
As mentioned in PR #346, PHPC and HHVM differ in their Decimal128 exception message for an invalid string. For the ObjectID message, both drivers are consistent, but the error message is brief and does not report the invalid value.
I propose we change these messages to the following:
- Error parsing Decimal128 string: %s
- Error parsing ObjectID string: %s
HHVM currently refers to "The argument", which I'd like to remove, since this error will be common to __construct(), __set_state(), and __wakeup(). It is merely a field instead of a proper method argument for the last two functions.