Opened 11 years ago

Last modified 8 years ago

#58 confirmed task

Remove LogNotifier from API

Reported by: ibboard Owned by:
Priority: minor Milestone: WarFoundry API 0.2
Component: WarFoundry-API Version: Unknown or N/A
Keywords: papercut Cc:
Blocked By: Blocking:

Description

While the initial idea of having some way of raising "log this" messages from the API code without requiring specific logging (e.g. Log4Net), it is actually a bit of an ugly hack. It should be replaced by specific events that can be listened to and logged or otherwise handled as appropriate.

Change History (11)

comment:1 Changed 11 years ago by ibboard

Owner: ibboard deleted

Remove owner so that the team can more easily see bugs. Using default owners seems good at first but is bad practice when a team can be working on any part of the project.

comment:2 Changed 10 years ago by ibboard

(In r153) Re #58: Remove LogNotifier from API

  • Remove dependency on IBBoard.Log4Net, since use in API will be removed as part of this ticket

comment:3 Changed 10 years ago by ibboard

(In r175) Re #58: Remove log notifier

  • Remove notified from XML Race Factory

comment:4 Changed 10 years ago by ibboard

Milestone: → WarFoundry 0.2

Move some tickets to v0.2 milestone

comment:5 Changed 10 years ago by ibboard

Status: new → confirmed

Move tickets to new "confirmed" status to show that they've been checked

comment:6 Changed 10 years ago by ibboard

Keywords: papercut added
released: → no
Version: → Unknown or N/A

comment:7 Changed 9 years ago by ibboard

Milestone: WarFoundry 0.2 → WarFoundry API 0.2

Separate API milestone from UI milestone

comment:8 Changed 9 years ago by ibboard

Priority: major → minor

Lower priority - it should be a simple task, but it isn't that serious if it is still in there.

comment:9 Changed 8 years ago by IBBoard <dev@…>

In cbeee87dc2d3a07a63eb3b1ddf6ad3dd43857a99/IBBoard.WarFoundry.API:

Re #58: Remove LogNotifier from API

  • Remove LogNotifier from API - mostly unnecessary logging

Also:

  • Formatting auto-corrected
  • LoadFile() "try...catch {silently dispose}" removed. Code shouldn't be throwing those errors and needs to be handled elsewhere if it does

comment:10 Changed 8 years ago by IBBoard <dev@…>

In 55c94d438a4b6db3eb59d2bb7ae5bfc54c67fa7b/IBBoard.WarFoundry.GUI.WinForms:

Re #58: Remove LogNotifier from API

  • Add logging moved from API

comment:11 Changed 8 years ago by IBBoard <dev@…>

In 83b19b57cba13926c06f201fc0981e7fb955b85f/IBBoard.WarFoundry.GUI.GTK:

Re #58: Remove LogNotifier from API

  • Move one of the semi-useful bits of logging to the GTK GUI
Note: See TracTickets for help on using tickets.