# |
Apr 21st 2017, 18:57 |
nemmons |
lol why is the PR on *my* fork of Footprint? |
# |
Apr 21st 2017, 18:57 |
dereuromark |
should be part of your deployment script |
# |
Apr 21st 2017, 18:55 |
chrisso |
Have anyone experienced `bin/cake: denied permission` when you pull changes from repo to prod. server? So annoying to chmod bin/cake after every pull. |
# |
Apr 21st 2017, 18:25 |
junior |
@admad , created PR at: https://github.com/nemmons/Footprint/pull/1 |
# |
Apr 21st 2017, 17:24 |
hmic |
:-( |
# |
Apr 21st 2017, 17:24 |
hmic |
it's a shame! |
# |
Apr 21st 2017, 17:23 |
hmic |
admad: have you seen his paste? |
# |
Apr 21st 2017, 17:20 |
admad |
"no wonder this spits on you!" LOL |
# |
Apr 21st 2017, 16:25 |
Dinossaur |
@hmic thanks for trying... I'll go back to 3.1, for now... |
# |
Apr 21st 2017, 16:23 |
watermark |
I know you're right, but not sure being right matters here |
# |
Apr 21st 2017, 16:23 |
hmic |
thats the proper solution to the problem. |
# |
Apr 21st 2017, 16:22 |
watermark |
no, that is logical, it won't work |
# |
Apr 21st 2017, 16:21 |
hmic |
set the clock correct and set the alarms 10 mins before the events |
# |
Apr 21st 2017, 16:21 |
watermark |
so, slightly off topic, my boss just said we cannot use two factor totp because he wants to set the clock on his phone 10 minutes fast so he's not late to things. What do? |
# |
Apr 21st 2017, 16:20 |
hmic |
*shudder* |
# |
Apr 21st 2017, 16:20 |
hmic |
there is lots of things that need to be fixed |
# |
Apr 21st 2017, 16:19 |
Dinossaur |
Literal queries was done on my first encounter with CakePHP. I just had not learn ORM then. Not escaping was a blunder. I'll fix it |
# |
Apr 21st 2017, 16:15 |
hmic |
are you kiding me? |
# |
Apr 21st 2017, 16:15 |
hmic |
$conn->execute($query); |
# |
Apr 21st 2017, 16:15 |
hmic |
); |
# |
Apr 21st 2017, 16:15 |
hmic |
$_SERVER['clienteDominio'] |
# |
Apr 21st 2017, 16:15 |
hmic |
$req->query['Usuario_id'], |
# |
Apr 21st 2017, 16:15 |
hmic |
$entity->id, |
# |
Apr 21st 2017, 16:15 |
hmic |
you use literal queries with user data, not escaping it at all, not binding the values at least and not using the orm in the same function for no reason? $query = sprintf("insert ignore into sis_usuario_perfil (Perfil_id,Usuario_id,Cliente_id) values (%d,%d,%d)", |
# |
Apr 21st 2017, 16:15 |
Dinossaur |
sorry... I'm not sure what is "super globals from a tables callback function"... |
# |
Apr 21st 2017, 16:14 |
Dinossaur |
https://thepasteb.in/p/vghOxz56BZof3 |
# |
Apr 21st 2017, 16:14 |
hmic |
no wonder this spits on you! |
# |
Apr 21st 2017, 16:14 |
hmic |
you are setting super globals from a tables callback function? |
# |
Apr 21st 2017, 16:14 |
hmic |
$_SERVER['ENTITY'] = $entity; |
# |
Apr 21st 2017, 16:14 |
hmic |
you know what, that code is really really bad! check this: if (isset($req->query['Usuario_id'])) { |
# |
Apr 21st 2017, 16:12 |
hmic |
can you show the query from UsersController line 147? |
# |
Apr 21st 2017, 16:08 |
Dinossaur |
@hmic i made the changes, but the error persists as before. |
# |
Apr 21st 2017, 16:06 |
hmic |
probably you just need to add the behaviour after setting the tablename and alias, so move that line further down |
# |
Apr 21st 2017, 16:05 |
hmic |
additionally call parent::initialize in the end of that function, not the beginning |
# |
Apr 21st 2017, 16:04 |
hmic |
Dinossaur, do you need the setAlias('users') really? - i dont use that. and if you do, make sure it reads "Users" not "users" IMHO |
# |
Apr 21st 2017, 16:04 |
birdy247 |
import charges |
# |
Apr 21st 2017, 16:03 |
hmic |
does it matter? costs 50$ shipping and a week of waiting |
# |
Apr 21st 2017, 16:02 |
birdy247 |
Im UK |
# |
Apr 21st 2017, 16:02 |
birdy247 |
All in the states |
# |
Apr 21st 2017, 16:02 |
hmic |
these are dead cheap! |
# |
Apr 21st 2017, 16:02 |
birdy247 |
I think id rather just buy new for ease of life |