error in entity_property_verbatim_get

This should probably get fixed anyway.

@kinmen, Please find the updated patch, to resolve I've added required arguments on entity_property_verbatim_get function. Hope this will helps.

Thanks

Based on #5.

Adjusting tags, seems worth a bump too ;)

entity_metadata_verbatim_get() has been deprecated for more than 10 years but still exists as a wrapper around entity_property_verbatim_get() so that anyone still calling the old function will actually be running the new function. Drupal's policy on API changes states we shouldn't make an API change between minor point releases, so removing entity_metadata_verbatim_get() entirely during the D7 lifecycle is not an option.

The problem is that the replacement function entity_property_verbatim_get() has two additional required parameters which the wrapper doesn't know about. More modern versions of PHP will complain that the wrapper is calling entity_property_verbatim_get() with the wrong number of parameters, and that is a valid complaint. We want to be able to use Entity API on more modern versions of PHP, and we want to maintain backwards compatibility as much as possible for the many existing sites that might be calling the old deprecated function.

The patch does this:

 function entity_metadata_verbatim_get($data, array $options, $name) {
-  return entity_property_verbatim_get($data, $options, $name);
+  return entity_property_verbatim_get($data, $options, $name, $type, $info);
 }

While this DOES provide the correct number of parameters to entity_property_verbatim_get(), the last two parameters, $type and $info, are undefined variables so this patch will ALSO generate complaints. Because these are required parameters, I think it's better to pass actual values - NULL instead of $type and array() instead of $info. These are both valid values/types for those parameters, and will not cause any change in operation to entity_property_verbatim_get() or entity_property_verbatim_get().

ya, last two params, pass NULL and array() should be ok.

Committed #8.