MyAlerts compatibility issues

[Sama34]

Hi, the upcoming version of OUGC Awards allows for MyAlerts integration, while testing your plugin I found that you insert the award directly into the DB.

I'd recommend to use the provided method so that all awards are inserted equally, for any insert hooks to run, and to ignite the alert code within the plugin.

The following change did it for me and should work even for the current official released package. From:

Newpoints-Awards/Upload/inc/plugins/newpoints/newpoints_awards.php

Lines 335 to 340 in f9e617f

	$db->insert_query('ougc_awards_users', [
	'uid' => (int) $mybb->user['uid'],
	'aid' => $aid,
	'reason' => $db->escape_string($award['reason']),
	'date' => TIME_NOW
	]);

To:

			global $awards;
			$awards->give_award($awards->get_award($aid), $mybb->user, $award['reason']);

[Sama34]

Oh, I also found that the plugin doesn't allows for more than 1 award per user. I can't recall if this is forced within the current package, but for the coming update the plugin allows multiple granting. You might want to look into it.

[chack1172]

Thank you for reporting it. I'll check it and fix these problems in the next version.