| # |
Jul 12th 2017, 15:36 |
jlroberts |
but why is it not global in Installer.php is the question |
| # |
Jul 12th 2017, 15:35 |
 jeremyharris |
if they were in a namespace, we’d need to do something like Cake\debug($var) everytime. they’re global for convenience |
| # |
Jul 12th 2017, 15:35 |
jeremyharris |
remove the function_exist calls and you’ll see everything works just fine in a typical request (it is not loaded multiple times) |
| # |
Jul 12th 2017, 15:35 |
slackebot |
~ Ferrero Roche |
| # |
Jul 12th 2017, 15:35 |
slackebot |
Command sent from Slack by neon1024: |
| # |
Jul 12th 2017, 15:34 |
neon1024 |
“Monsieur, with this Composer you are really spoiling us” |
| # |
Jul 12th 2017, 15:34 |
jeremyharris |
no I think those just exist so you can overwrite them if you want, or to prevent collision in the global namespace |
| # |
Jul 12th 2017, 15:34 |
jlroberts |
but thats the point, everytime the autoloader fires, it has to include the functions.php file, hence if (!function_exists(.... in functions.php |
| # |
Jul 12th 2017, 15:33 |
jeremyharris |
just typical PHP stuff here |
| # |
Jul 12th 2017, 15:33 |
jeremyharris |
those functions are in the global namespace, so cake just uses require to bring them into the application |
| # |
Jul 12th 2017, 15:33 |
jeremyharris |
but it doesn’t and can’t, as those functions aren’t in a namespace, so the autoloader couldn’t find it even if it wanted to |
| # |
Jul 12th 2017, 15:32 |
jlroberts |
ok, so somewhere, there is an autoloader, that is autoloading classes, from what I understand, that same autoloader needs to autoload the functions.php file |
| # |
Jul 12th 2017, 15:31 |
jeremyharris |
it’s `require_once` by cake during bootstrap |
| # |
Jul 12th 2017, 15:31 |
jeremyharris |
I mean, it’s not autoloaded by PHP |
| # |
Jul 12th 2017, 15:31 |
jeremyharris |
it’s not even autoloaded by cake |
| # |
Jul 12th 2017, 15:30 |
jeremyharris |
I think that’s the best way. It’s sort of how cake does it anyway during bootstrap |
| # |
Jul 12th 2017, 15:30 |
jlroberts |
heh =] i just mean, we are using namespace's, use's, and autoloaders in CakePHP apps... |
| # |
Jul 12th 2017, 15:30 |
neon1024 |
Mind blown. |
| # |
Jul 12th 2017, 15:30 |
neon1024 |
:P |
| # |
Jul 12th 2017, 15:30 |
neon1024 |
Coding PHP isn’t the right way?! |
| # |
Jul 12th 2017, 15:29 |
jlroberts |
neon1024: thats exactly what I need, but I dont think thats the right way to do it |
| # |
Jul 12th 2017, 15:29 |
neon1024 |
Sounds like you need `require($myPhpFile)` |
| # |
Jul 12th 2017, 15:29 |
jlroberts |
the only place I see it is in autoload_files.php @generated by composer |
| # |
Jul 12th 2017, 15:27 |
jlroberts |
Here is my github issue for more details, https://github.com/cakephp/app/issues/525 |
| # |
Jul 12th 2017, 15:27 |
jlroberts |
ok fellas, I got a question for yall, I am trying to dispatch shells from app/src/Console/Installer.php, but it appears that the cakephp/src/Core/functions.php file is not autoloaded, however, the use statements in Installer.php is autoloaded... anyone know how cakephp/src/Core/functions.php is loaded outside of Installer.php? |
| # |
Jul 12th 2017, 15:26 |
jeremyharris |
:+1: lemme know! |
| # |
Jul 12th 2017, 15:26 |
ericadeefox |
ok backpedal, I'm gonna give rewriting these a shot before I gist em |
| # |
Jul 12th 2017, 15:26 |
jeremyharris |
also, make sure there’s only one `$this->post()` and friends during each test case |
| # |
Jul 12th 2017, 15:25 |
jeremyharris |
that’s usually how integration tests look. maybe gist one so I can see :slightly_smiling_face: |
| # |
Jul 12th 2017, 15:25 |
ericadeefox |
does this mean I should rewrite my tests so that they _start_ with requests? like `$this->post()` or `$this->get()` at the beginning, so the session object is created right off the jump? |
| # |
Jul 12th 2017, 15:24 |
ericadeefox |
so in testing, since the session object isn't created until a request is--oh, wait |
| # |
Jul 12th 2017, 15:23 |
ericadeefox |
^that's basically what's happening. the plugin checks to see if a session has been started, then 500s and throws an error when it isn't. |
| # |
Jul 12th 2017, 15:23 |
jeremyharris |
perhaps an FB class is instantiated early and checks the session? |
| # |
Jul 12th 2017, 15:22 |
jeremyharris |
you shouldn’t need session_start anywhere. other integration tests work successfully using the session, no? |
| # |
Jul 12th 2017, 15:22 |
jeremyharris |
ok so this is the process: `$this->session` just stores what you want to write, `$this->post()` starts by building a request, it sees you want to write session data, so it creates a session object and `$session->write()`s to it. `$session->write()` sees the session isn’t started yet, so it starts it for you. the request is then dispatched |
| # |
Jul 12th 2017, 15:19 |
ericadeefox |
hmm... my tests don't work unless I use `session_start()` at the top of the first test. but then using that, a few of the tests still don't work, because they have to write stuff using `$this->session()`. idk if there is any way to make them compatible. |
| # |
Jul 12th 2017, 15:17 |
jlroberts |
word |
| # |
Jul 12th 2017, 15:17 |
jeremyharris |
that’s why `$this->session` exists, to write into the session when the request is actually built |
| # |
Jul 12th 2017, 15:17 |
ericadeefox |
ahhh. ::nod:: |
| # |
Jul 12th 2017, 15:17 |
jeremyharris |
no it’s not, because the request isn’t built until the test method (e.g., get, post) executes |
| # |
Jul 12th 2017, 15:16 |
ericadeefox |
I mean, is it _possible_ to write session data using `$this->request->session()->write()` in integration tests? because I just keep getting the error message `Undefined property: App\Test\TestCase\Controller\EventsControllerTest::$request` when I do this. |