| # |
Mar 6th 2018, 00:24 |
 jeremyharris |
Np! |
| # |
Mar 6th 2018, 00:24 |
mikesmoniker |
Coolio. Thanks! |
| # |
Mar 6th 2018, 00:24 |
jeremyharris |
Interesting, yeah open up an issue and I'll check it out tomorrow :+1: |
| # |
Mar 6th 2018, 00:23 |
jeremyharris |
(Sorry on my phone) |
| # |
Mar 6th 2018, 00:23 |
mikesmoniker |
But the second test fails. |
| # |
Mar 6th 2018, 00:23 |
mikesmoniker |
The first test passes, which yeah, is basically as if we aren’t even using the trait (b/c we aren’t). |
| # |
Mar 6th 2018, 00:23 |
jeremyharris |
Oh I see the second test does |
| # |
Mar 6th 2018, 00:22 |
jeremyharris |
I don't understand. The test cases reset the entity class, so the lazy loader shouldn't be involved at all |
| # |
Mar 6th 2018, 00:22 |
mikesmoniker |
Should I just go ahead and open an issue? I haven’t had time to dig into what’s up, but hopefully the tests are enough to communicate the perceived problem. |
| # |
Mar 6th 2018, 00:21 |
mikesmoniker |
Thinking about opening an issue on lazyload. The trait seems to interfere with how by reference array functions work with relationships. I added a couple tests to demonstrate the issue: https://github.com/themrwilliams/cakephp-lazyload/commit/185aa5a4befb3b4a3e016ea79d5bb9a6894d4c2d |
| # |
Mar 6th 2018, 00:20 |
jeremyharris |
Nah, but what's up? |
| # |
Mar 6th 2018, 00:19 |
mikesmoniker |
@jeremyharris any chance you’re lurking? |
| # |
Mar 5th 2018, 23:28 |
ricksaccous |
$this->request->is(['pdf']) is not a thing, lol |
| # |
Mar 5th 2018, 23:27 |
ricksaccous |
File uploaded https://cakesf.slack.com/files/U8LQ47EPR/F9K3W2M37/appcontroller_php.php / https://slack-files.com/T053DPNCM-F9K3W2M37-ed9cf83a5b |
| # |
Mar 5th 2018, 23:26 |
ricksaccous |
and I stopped setting the viewClass in beforeFilter... it seems that |
| # |
Mar 5th 2018, 23:26 |
ricksaccous |
I removed $this->RequestHandler->renderAs($this, 'pdf', ['attachment' => 'filename.pdf']); this line |
| # |
Mar 5th 2018, 23:26 |
ricksaccous |
I kind of sort of got it working |
| # |
Mar 5th 2018, 23:09 |
ricksaccous |
so yeah, order matters, heh |
| # |
Mar 5th 2018, 23:09 |
ricksaccous |
when I request .pdf |
| # |
Mar 5th 2018, 23:09 |
ricksaccous |
the viewclass for the plugin actually loads now |
| # |
Mar 5th 2018, 23:09 |
ricksaccous |
it did fix something though |
| # |
Mar 5th 2018, 23:08 |
ricksaccous |
nah that didn't fix the path |
| # |
Mar 5th 2018, 23:06 |
ricksaccous |
going to try it first before i test it outside the plugin |
| # |
Mar 5th 2018, 23:06 |
savant |
did it? |
| # |
Mar 5th 2018, 23:06 |
ricksaccous |
sheeeeeeeeeet |
| # |
Mar 5th 2018, 23:06 |
ricksaccous |
if a plugin depends on another you'd have to loa.... |
| # |
Mar 5th 2018, 23:06 |
ricksaccous |
would order of plugins matter? |
| # |
Mar 5th 2018, 23:05 |
ricksaccous |
wait |
| # |
Mar 5th 2018, 23:02 |
ricksaccous |
but yeah it's having problems here it seems.... not sure what's triggering it |
| # |
Mar 5th 2018, 23:01 |
ricksaccous |
I think I'll test it outside the plugin and see if it works and come back maybe |
| # |
Mar 5th 2018, 23:01 |
ricksaccous |
CakePdf plugin* |
| # |
Mar 5th 2018, 23:01 |
ricksaccous |
my own plugin |
| # |
Mar 5th 2018, 23:01 |
ricksaccous |
i don't think so but what i am doing is trying to use the CakePdf from within a plugin |
| # |
Mar 5th 2018, 23:01 |
savant |
thats interesting, are you changing your search paths somewhere somehow? |
| # |
Mar 5th 2018, 23:00 |
ricksaccous |
File uploaded https://cakesf.slack.com/files/U8LQ47EPR/F9KQXUW3Y/-.txt / https://slack-files.com/T053DPNCM-F9KQXUW3Y-c9f9c6b1d6 |
| # |
Mar 5th 2018, 22:59 |
ricksaccous |
so I think I've narrowed the problem somewhat, it's looking for a template within a plugin, but it is omitting the Controller name from the template it's looking for |
| # |
Mar 5th 2018, 22:50 |
savant |
https://newrelic.com/why-new-relic/customers <-- list of their customers |
| # |
Mar 5th 2018, 22:49 |
savant |
I personally dont have any issues with that |
| # |
Mar 5th 2018, 22:49 |
savant |
yeah, so? |
| # |
Mar 5th 2018, 22:49 |
meder |
.. new relic that is |
| # |
Mar 5th 2018, 22:49 |
meder |
they have some sort of disclaimer |