Ticket #508 (reopened patch)

Opened 5 years ago

Last modified 4 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 5 years ago.
Patch to rename to JSONReader and JSONWriter

Change History

Changed 5 years ago by bsdz

Patch to rename to JSONReader and JSONWriter

Changed 5 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 5 years ago by whiteknight

  • milestone 1.5 deleted

Changed 4 years ago by coke

compilers/json is gone, use data_json instead.

Changed 4 years ago by bacek

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

So, resolving ticket.

Changed 4 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.