| # |
Jul 17th 2009, 17:35 |
gwoo |
because it seems to only take into account these tested cases |
| # |
Jul 17th 2009, 17:34 |
gwoo |
the fix though i think needs some work |
| # |
Jul 17th 2009, 17:34 |
gwoo |
i see the issue it points out |
| # |
Jul 17th 2009, 17:34 |
Phally |
*patch |
| # |
Jul 17th 2009, 17:34 |
Phally |
gwoo: how about the PaginatorHelper, do you agree on the path? |
| # |
Jul 17th 2009, 17:33 |
gwoo |
ok cool |
| # |
Jul 17th 2009, 17:33 |
Phally |
yes, i know what you mean |
| # |
Jul 17th 2009, 17:33 |
gwoo |
Phally: did I explain it well enough? |
| # |
Jul 17th 2009, 17:33 |
Phally |
very well, i will refactor it |
| # |
Jul 17th 2009, 17:32 |
gwoo |
$controller gets passed to the callbacks because those are the only places where it should be needed |
| # |
Jul 17th 2009, 17:32 |
gwoo |
it is hard to know how much of "controller" is really used when you just set a property to the value coming in from the callbacks |
| # |
Jul 17th 2009, 17:32 |
gwoo |
and set it to a member of the current class |
| # |
Jul 17th 2009, 17:31 |
gwoo |
to get it |
| # |
Jul 17th 2009, 17:31 |
gwoo |
use the initialize or startup method |
| # |
Jul 17th 2009, 17:31 |
gwoo |
if you need data from teh Auth component |
| # |
Jul 17th 2009, 17:31 |
gwoo |
meaning that the original callbacks have not been used properly |
| # |
Jul 17th 2009, 17:31 |
gwoo |
to me that is bad form |
| # |
Jul 17th 2009, 17:31 |
gwoo |
Phally: also try to avoid set $this->__controller |
| # |
Jul 17th 2009, 17:30 |
gwoo |
in the second case |
| # |
Jul 17th 2009, 17:30 |
gwoo |
Phally: use isAuthorized |
| # |
Jul 17th 2009, 17:29 |
pointlessjon |
has anyone considered shell tasks as a category for articles? |
| # |
Jul 17th 2009, 17:29 |
Phally |
gwoo: so i extracted it as a method |
| # |
Jul 17th 2009, 17:29 |
Phally |
gwoo: it is called twice |
| # |
Jul 17th 2009, 17:28 |
gwoo |
why not just put that code in isAuthorized? |
| # |
Jul 17th 2009, 17:28 |
gwoo |
Phally: what is the purpose behind the private __check method? |
| # |
Jul 17th 2009, 17:27 |
candybar |
) |
| # |
Jul 17th 2009, 17:27 |
Phally |
candybar: oh, ok, very well then, thanks for paying attention though :) |
| # |
Jul 17th 2009, 17:26 |
candybar |
no, i was having a moment, there is no concern |
| # |
Jul 17th 2009, 17:25 |
Phally |
candybar: just the person i was looking for, i heard you shared your concerns regarding the permission system we had planned for the bakery? |
| # |
Jul 17th 2009, 17:10 |
Phally |
pointlessjon: http://thechaw.com/bakery/source/branches/usersplugin |
| # |
Jul 17th 2009, 17:07 |
pointlessjon |
what are you working on Phally? |
| # |
Jul 17th 2009, 17:07 |
Phally |
... |
| # |
Jul 17th 2009, 17:07 |
pointlessjon |
good, get back to work! |
| # |
Jul 17th 2009, 17:05 |
Phally |
yes |
| # |
Jul 17th 2009, 17:03 |
pointlessjon |
any of you working on any bakery related tasks atm? |
| # |
Jul 17th 2009, 17:02 |
proloser |
lol no love for techno |
| # |
Jul 17th 2009, 17:01 |
dennishennen |
techno-geek: I'm new in this channel myself, but I can say the chatter is distracting. You'll end up just getting put in everyone's ignore file and no one will hear you when you have a legit question. |
| # |
Jul 17th 2009, 16:56 |
proloser |
!log |
| # |
Jul 17th 2009, 16:54 |
pointlessjon |
gwoo: totally reasonable. ADmad: with all due respect, you should try and be a little more diplomatic |
| # |
Jul 17th 2009, 16:54 |
techno-geek |
well I apologize then. I was just responding to the conversation because I enjoy all of your company. I just wish that ADmad and alkemann wouldn't be riding our cases so much, especially when we are discussing legit topics |
| # |
Jul 17th 2009, 16:53 |
gwoo |
there is already plenty of that with the way ADmad and alkemann talk about stuff |