trigger() checks type rather than event for 'modify', 'delete'

[fbennett]

While sorting out a UI issue in the multilingual branch related to notifier events, I noticed that the trigger() function in notifier.js checks the type argument for the strings 'modify' and 'delete', and issues a warning if the extraData argument is not also supplied with these.

A debugging trace, and the content of the _types array, showed that these strings appear on the event argument, but not on type.

I'm not sure if it's important at all, but the attached patch causes the function to check the event argument instead.

[dstillman]

In [10609]:

Fixes #1796, trigger() checks type rather than event for 'modify', 'delete'

[dstillman]

In [10639]:

Addresses #1796, trigger() checks type rather than event for 'modify', 'delete'

Just take out check for extraData