Warning: Can't synchronize with repository "(default)" (Unsupported version control system "svn": No module named svn). Look in the Trac log for more information.

Ticket #2212 (closed task: fixed)

Opened 10 years ago

Last modified 10 years ago

[patch] Eliminate TurboJson and use plain simplejson.

Reported by: jorge.vargas Owned by: jorge.vargas
Priority: high Milestone: 2.1a1
Component: TurboGears Version: trunk
Severity: normal Keywords:
Cc:

Description

Original discussion  tg trunk list

Goals

  • reduce the complexity of TG
  • reduce the amount of dependencies (7 packages are pulled in by TurboJson?, since TW doesn't uses it anymore)
  • in theory increase speed. (less layers)

Attachments

noTJ.patch Download (3.8 KB) - added by jorge.vargas 10 years ago.
Updated cleaned up patch, it drops buffet renderer but still uses turbojson (if installed)
noTJ.2.patch Download (3.8 KB) - added by jorge.vargas 10 years ago.
using a better try/catch, I'm impress noone has wondered why the check is inverted, this could be a major bug :)

Change History

comment:1 Changed 10 years ago by mramm

  • Milestone changed from 2.0b6 to 2.1

comment:2 Changed 10 years ago by mramm

This can be done in 2.1, but it much too large of a change for 2.0 at this point.

Also note that turbojson also automatically handles SA mapped objects and has a well understood override mechanism, which this no tj patch does not.

comment:3 Changed 10 years ago by jorge.vargas

  • Owner changed from faide to jorge.vargas

Changed 10 years ago by jorge.vargas

Updated cleaned up patch, it drops buffet renderer but still uses turbojson (if installed)

Changed 10 years ago by jorge.vargas

using a better try/catch, I'm impress noone has wondered why the check is inverted, this could be a major bug :)

comment:4 Changed 10 years ago by percious

  • Milestone changed from 2.1 to 2.1a1

comment:5 Changed 10 years ago by percious

jorge, is this complete?

comment:6 Changed 10 years ago by percious

  • Status changed from new to closed
  • Resolution set to fixed
Note: See TracTickets for help on using tickets.