<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2014-05-15 20:19 GMT+02:00 Nicolas Chauvat <span dir="ltr"><<a href="mailto:nicolas.chauvat@logilab.fr" target="_blank">nicolas.chauvat@logilab.fr</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<div class=""><br>
On Thu, May 15, 2014 at 09:49:49AM +0200, Aurélien Campéas wrote:<br>
> On 15/05/2014 09:25, Denis Laxalde wrote:<br>
> > aurélien campéas a écrit :<br>
> >> Whatever the future holds for _cw, I'd very much like it were re-cast as<br>
> >> just 'cw'.<br>
> ><br>
> > At this point, using a descriptive name wouldn't hurt.<br>
><br>
> Yes it would. "cubicweb_god_object" is descriptive but too long.<br>
<br>
</div>1) _cw is short and that is a quality for a name that is used a lot<br>
   cw would be shorter, but make it less obvious that it is a private<br>
   attribute<br></blockquote><div><br><br></div><div>I think it could be a local parameter, as in:<br><br><br></div><div>  class HelloView(View):<br></div><div><div>      __regid__ 'hello'<br></div><div>      __select__ = is_instance('IndexCard') & title('summary')<br>
</div><br>      def render(self, cw, rset, *args, **kw):<br></div><div>          with cw.t.h1(_class=self.__regid__):<br></div><div>              cw.w(cw._('Hello, World, says %s' % <a href="http://cw.appli.name">cw.appli.name</a>))<br>
</div><div>          if cw.cnx.is_anonymous:<br></div><div>              cw.w(cw.t.p('Register now and get a free cw !'))<br></div><div>          cw.view('blogentry', rset)<br><br></div><div>This extreme version is a bit too cw, cw cw cw, cw cw.<br>
</div><div>In the case of a render method, it makes sense, for use<br>frequency reasons, to feed t, _ and w as parameters:<br><br><div>  class HelloView(View):<br></div><div>      __regid__ 'hello'<br></div><div>      __select__ = is_instance('IndexCard') & title('summary')<br>
</div><div><br>      def render(self, cw, w, t, _, rset, *args, **kw):<br></div><div>          with t.h1(_class=self.__regid__):<br></div><div>              w(_('Hello, World, says %s' % <a href="http://cw.appli.name">cw.appli.name</a>))<br>
</div><div>          if cw.cnx.is_anonymous:<br>              w(t.p('Register now and get a free cw !'))<br></div><div>          cw.view('blogentry', rset)<br><br><br></div><div>This clicks.<br></div><br></div>
<div>Regards,<br></div><div>Aurélien.<br></div></div></div></div>