Ticket #1651 (closed feature: fixed)

Opened 5 years ago

Last modified 5 years ago

Add OpenLayers.Console.userError

Reported by: tcoulter Owned by: crschmidt
Priority: critical Milestone: 2.7 Release
Component: Console Version: 2.6
Keywords: Cc:
State: Complete

Description

Talk w/ tschaub, crschmidt, and euzuro resulted in all OL errors (currently alerts) should go through OpenLayers.Console.error(). On top of that, OpenLayers.Console.error()'s default behavior should alert the message passed to it, though it should allow for easy overriding to show prettier/different error messages.

Attachments

usererror.patch Download (15.2 KB) - added by tcoulter 5 years ago.
Adding support w/ example for a single interface for showing user firendly error messages.

Change History

Changed 5 years ago by tcoulter

  • owner changed from tcoulter to crschmidt
  • summary changed from OpenLayers.Console.error to Add OpenLayers.Console.userError

Turns out OpenLayers.Console.error() is used many places elsewhere; added OpenLayers.Console.userError().

Assigning to crschmidt to review (committing patch in a sec).

Changed 5 years ago by tcoulter

Adding support w/ example for a single interface for showing user firendly error messages.

Changed 5 years ago by crschmidt

  • status changed from new to closed
  • state set to Complete
  • resolution set to fixed

(In [7675]) Patch from funky_c to move all alert()s in the library to a single OpenLayers.Console.userError function. This allows for easy overriding of this function for people who need to report errors at an application level (or hiding these errors entirely), with the same default behavior as previous releases. Patch from funky_c, r=me, (Closes #1651)

Changed 5 years ago by euzuro

  • component changed from general to Console
Note: See TracTickets for help on using tickets.