Ticket #508 (reopened patch)

Opened 11 years ago

Last modified 10 years ago

JSON libraries should be renamed to something less confusing

Reported by: bsdz Owned by:
Priority: normal Milestone:
Component: library Version:
Severity: medium Keywords:
Cc: Language:
Patch status: obsolete Platform:

Description

There are three JSON.pbc files currently:-

* compilers/json/JSON.pbc - for reading JSON. * library/JSON.pbc - for writing JSON. * library/Config/JSON.pbc for reading and writing config files containing JSON.

The supplied patch renames the purely read/write ones to library/JSONReader.pbc and library/JSONWriter.pbc.

However, what this patch misses is leaving a stub PBCs to manage a proper deprecation cycle.

Attachments

json.patch Download (28.8 KB) - added by bsdz 11 years ago.
Patch to rename to JSONReader and JSONWriter

Change History

Changed 11 years ago by bsdz

Patch to rename to JSONReader and JSONWriter

Changed 11 years ago by allison

Rather than putting in deprecation stubs, we'll just enter a deprecation notice now for the old library names (done in 37768) and make the change in 1.5.

For consistency and clarity, let's move these to library/JSON/Reader.pbc, library/JSON/Writer.pbc and library/JSON/Config.pbc. (With the class names ['JSON'; 'Reader'] etc).

Changed 11 years ago by whiteknight

  • milestone 1.5 deleted

Changed 10 years ago by coke

compilers/json is gone, use data_json instead.

Changed 10 years ago by bacek

  • status changed from new to closed
  • resolution set to fixed
  • patch changed from new to obsolete

So, resolving ticket.

Changed 10 years ago by coke

  • status changed from closed to reopened
  • resolution fixed deleted

Original issue still not resolved.

Note: See TracTickets for help on using tickets.