response.view as a decorator

[samuel bonill]

hi all.

i'm trying implement response.view as a decorator, but not work my code:
for example i wanna do it:

@View('path/sub_path/view.html')
def index()
      return locals()

in modules i have :  decorator.py

from gluon.globals import Response
response = Response()

class Error(Exception):
    ''' Error if path != str''

class View(object):
      
    def __init__(self, path):
        if isinstance(path, str):
            self.path = path
        else:
            raise Error('{} no es una cadena'.format(path))
           
    def __call__(self, fn):
        def _call():
           ## fn.func_globals[response.view] = self.path
           respomse.view = self.path
           return fn()
        return _call

in default.py :

from decorador import View

@View('layout.html')
def exampler():
    '''para llamar una vista de cualquier directorio'''
    return locals()

but not found !!!.... why ?

not want to put on each controller response.view = 'example'.... ! help !!

[Niphlod]

uhm. I'm totally ok for decorators that save lots of typing.
What's so different between
@View('whatever')
and
response.view = 'whatever'
?

[Niphlod]

BTW, I'm a fan of functional.

from gluon import current
def outer(view):
    def wrap(f):
        def wrapped_f():
            current.response.view = view
            return f()
        return wrapped_f
    return wrap

@outer('path/to/view')
def index():
      return locals()

[samuel bonill]

thanks... the error is in import gluon.globals :

from gluon.globals import Response
response = Response() # import error

now:

from gluon import current

response = current.response

class View(object):

       def __init__(self, path):
            ........

        def __call__(self, fn):

            def _call():

                 response.view = self.path
                ......

                 ...... etc....

now work....

2013/8/22 Niphlod <nip...@gmail.com>

--
 
---
You received this message because you are subscribed to a topic in the Google Groups "web2py-users" group.
To unsubscribe from this topic, visit https://groups.google.com/d/topic/web2py/ljp1IcabuSE/unsubscribe.
To unsubscribe from this group and all its topics, send an email to web2py+un...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

[Anthony]

from gluon import current

response = current.response

class View(object):

       def __init__(self, path):
            ........

        def __call__(self, fn):

            def _call():

                 response.view = self.path

No, don't do that. If you assign current.response to response at the top level in the module, it will not be updated on each request. Instead, in the _call() function, do current.response.view = self.path.

Anyway, I agree with Niphlod -- this seems like a completely gratuitous use of a decorator.

Anthony 

[samuel bonill]

thanks Anthony....

@View('path/to/file')

def example():

     return locals()

implement this decorator is based on my personal taste....

2013/8/22 Anthony <abas...@gmail.com>

from gluon import current

response = current.response

class View(object):

       def __init__(self, path):
            ........

        def __call__(self, fn):

            def _call():

                 response.view = self.path

No, don't do that. If you assign current.response to response at the top level in the module, it will not be updated on each request. Instead, in the _call() function, do current.response.view = self.path.

Anyway, I agree with Niphod -- this seems like a completely gratuitous use of a decorator.

Anthony 

--