OpenLayers OpenLayers

Ticket #1651 (closed feature: fixed)

Opened 2 years ago

Last modified 2 years ago

Add OpenLayers.Console.userError

Reported by: tcoulter Assigned to: 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 (15.2 kB) - added by tcoulter on 08/01/08 22:06:40.
Adding support w/ example for a single interface for showing user firendly error messages.

Change History

08/01/08 22:05:51 changed 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).

08/01/08 22:06:40 changed by tcoulter

  • attachment usererror.patch added.

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

08/01/08 22:21:50 changed 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)

09/05/08 06:24:15 changed by euzuro

  • component changed from general to Console.