there is no way to use txmongo in uimodule about cyclone HOT 12 CLOSED

I've been trying to fix this issue. However, it will require many changes before it can be fixed because of the nature of Twisted.
Still trying to minimize the impact on the way it works.

from cyclone.

I got this problem because I have a global ui mobule which requires db access.
I was trying to walk around this by injecting RequestHandler's "prepare", "render_string"..., but none of these function supports "inlineCallbacks". Still can't find a way.
I will really appreciate it if you kindly offer some suggestions.

from cyclone.

Yes, as I mentioned before, I've been trying many different solutions to this problem. What I don't want to do is to change the API in such a way that "render_string" return a deferred. That would require "yield self.render(..)", or even "text = yield self.render_string(..)".
I still don't have a final solution for this...

from cyclone.

here is my walk around:
since my project has several ui modules that appear all across the website, I have to do some db access works in the ui module.
I just:

1. write another handler that renders data which should have been rendered by ui module.
2. load data from the handler using ajax, and append it to the ui module html after page is loaded..
3. for other pages that need this ui module, just include the ajax script to them.
   it is not a good idea but works for now.

from cyclone.

Hi fiorix,

I added deferred support for templates, you can find the modified module here:
https://github.com/darkrho/txrho/blob/master/txrho/web.py#L245

It requires a new template node that you can find in templates.py source.

Usage example: {% yield modules.AsyncModule() %}

Didn't had time to document that feature yet :(
In my txrho repository I added other modifications that return the control to the reactor more often.

Regards,

from cyclone.

that's brilliant, darkrho.

from cyclone.

@darkrho can you get it merged into the current code base? would be great to have it before the release of cyclone 1.0

from cyclone.

@fiorix I would be able to cleanup the code and merge with the current database the last week of this month. Let me know if it's within the 1.0 release timeline.

from cyclone.

@darkrho yeah that works for me, thanks!

from cyclone.

Any updates on this? :)

from cyclone.

not yet

On 2012-04-26, at 4:10 AM, Njal Karevoll wrote:

Any updates on this? :)

---

Reply to this email directly or view it on GitHub:
#21 (comment)

Ship, ahoy! Hast seen the White Whale?

• Cap'n Ahab

from cyclone.

This require too many changes to the rendering structure. Honestly, I don't think there's an easy way to implement for now.
Please reopen this ticket whenever needed and we can put it on the table once again.

from cyclone.