#1516 Replace pylons.c usage

v1.0.0
closed
sf-4 (350)
General
nobody
2015-08-20
2011-02-16
Mark Ramm
No

Rather than use pylons.c and "magical_c" we should just make our own allura.c environ proxy and make our lives easier, and remove the word "magical" from our variable names ;)

Discussion

  • Rick Copeland - 2011-02-24
    • milestone: limbo --> mar-03
     
  • Rick Copeland - 2011-02-24

    I'm actually thinking we should use a 'streaming' registry manager and eliminate the magicalC -- streaming registry managers don't clean up their stacked object proxies (SOPs) until the iterator is exhausted, which is the behavior we want. TG by default uses a non-streaming registry manager, so the various SOPs are cleaned up when we return from the controller, before the template is rendered, which is a Bad Thing.

     
  • Mark Ramm - 2011-02-24

    Interesting, I didn't know that was a feature of paste SOP's. I'm not opposed to changing it, because I doubt many people are using SOP's in a way where it would make a difference.

     
  • Rick Copeland - 2011-02-24
    • size: --> 4
     
  • Rick Copeland - 2011-02-24
    • status: open --> in-progress
    • assigned_to: Rick Copéland
     
  • Rick Copeland - 2011-03-01

    A bunch of global-ish variables were switched to SOPs, our middleware was reordered, and we are now using the streaming registry to make sure our SOPs don't get cleaned up before the iterator is exhausted.

     
  • Rick Copeland - 2011-03-01
    • status: in-progress --> closed
     

Log in to post a comment.