evemon icon indicating copy to clipboard operation
evemon copied to clipboard

Sticky Industry Notifications

Open wvdvegt opened this issue 3 years ago • 1 comments

The current version accumulates industry notifications without ever removing them when jobs are delivered.

My fix for this is to add this method to GlobalNotificationCollection.cs:

internal void InvalidateIndustryCompleted(CCPCharacter character)
{
        Invalidate(new NotificationInvalidationEventArgs(character, NotificationCategory.IndustryJobsCompletion));
}

and call it from CharacterDataQuerying.cs as most other types (PI etc) do by adding a line:

// Invalidate previous notifications
EveMonClient.Notifications.InvalidateIndustryCompleted(target);

to the OnIndustryJobsUpdated() method (directly after the 'if (target != null)' statement.

wvdvegt avatar Oct 06 '21 12:10 wvdvegt

After testing this change does not seem to work properly

  • Accumulating is gone (positive)
  • But the notifications do not always appear at startup
  • And are not always removed on timer based updates.

wvdvegt avatar Oct 27 '21 07:10 wvdvegt