Resolution issue #3001

[JuanIMartinezB]

There was a reported issue with the "longid()" and "shortid()" methods of the Verbnet.py file. Basically, there are two types of ID, short id (eg. 32.5) and long id(eg. example-32.5). The methods involved change a short id to a long id and vice versa. The problem was that verbnet.longid("114-1") is supposed to return the long id "act-114-1", but it instead returns "114-1". This issue was solved by changing the Regex pattern to match the id types while still passing the tests for all other cases.

The change in the long id Regex pattern was:

_LONGID_RE = re.compile(r"([^\-\.]*)-([\d+.-]+)$") ------> LONGID_RE = re.compile(r"([A-Za-z]+)-([\d.-]+)$")

The new pattern tries to match an alphabetic sequence first (without considering empty sequences), then to match a minus sign "-" and finally the associated short id.

[tomaarsen]

Please have a look at the failing tests, e.g. these.