| # |
Sep 10th 2019, 08:35 |
 challgren |
20 pages and no arrays |
| # |
Sep 10th 2019, 08:32 |
challgren |
Ive gone through 8 pages so far and looks like everyone is doing string |
| # |
Sep 10th 2019, 08:31 |
admad |
Any genius actually using `$this->Html->image(['/webroot', 'img/', 'foo.png'])` deserves his app to be broken |
| # |
Sep 10th 2019, 08:30 |
challgren |
https://github.com/search?q=%24this-%3EHtml-%3Eimage%28%5Bandtype=Code |
| # |
Sep 10th 2019, 08:30 |
neon1024 |
But as you guys said, I can’t see how you’d even use an array to path a static asset, so impact might be minimal |
| # |
Sep 10th 2019, 08:29 |
neon1024 |
It feels wrong to knowingly break someones application |
| # |
Sep 10th 2019, 08:29 |
neon1024 |
I don’t feel comfortable intentionally breaking BC though |
| # |
Sep 10th 2019, 08:29 |
neon1024 |
:thinking_face: |
| # |
Sep 10th 2019, 08:28 |
neon1024 |
`$this->Html->image(['/webroot', 'img/', 'foo.png'])`? |
| # |
Sep 10th 2019, 08:28 |
neon1024 |
How would you get an image with an array then? |
| # |
Sep 10th 2019, 08:28 |
neon1024 |
Just so you have absolutly no idea what’s going on |
| # |
Sep 10th 2019, 08:27 |
neon1024 |
`@return mixed` |
| # |
Sep 10th 2019, 08:27 |
challgren |
ROFL |
| # |
Sep 10th 2019, 08:27 |
neon1024 |
With `@inheritDoc` |
| # |
Sep 10th 2019, 08:27 |
neon1024 |
Probably `__call()` |
| # |
Sep 10th 2019, 08:27 |
challgren |
Its that damm cake magic I tell yah! |
| # |
Sep 10th 2019, 08:27 |
neon1024 |
Magic! |
| # |
Sep 10th 2019, 08:27 |
admad |
how do think HtmlHelper::image() gets it's image path? :slightly_smiling_face: |
| # |
Sep 10th 2019, 08:27 |
neon1024 |
I tend to use `$this->Html->image('foo.png')` |
| # |
Sep 10th 2019, 08:26 |
neon1024 |
Yeah, but your ticket said Url::image right? |
| # |
Sep 10th 2019, 08:26 |
admad |
@neon1024 you don't use `HtmlHelper::image()`? |
| # |
Sep 10th 2019, 08:25 |
admad |
> But honestly how would you even build it with an array? Since its not linked to any controllers My point exactly |
| # |
Sep 10th 2019, 08:22 |
neon1024 |
I don’t use those methods @admad |
| # |
Sep 10th 2019, 08:21 |
challgren |
But honestly how would you even build it with an array? Since its not linked to any controllers |
| # |
Sep 10th 2019, 08:20 |
challgren |
So yeah not a breaking change for me really |
| # |
Sep 10th 2019, 08:19 |
challgren |
But $this->Url I only use 5 times |
| # |
Sep 10th 2019, 08:19 |
slackebot |
#3498db; border-radius: 5px; box-sizing: border-box; cursor: pointer; text-decoration: none; font-size: 14px; font-weight: bold; margin: 0; padding: 12px 25px; text-transform: capitalize; border-color: #3498db;">View Audit</a>``` |
| # |
Sep 10th 2019, 08:19 |
challgren |
```<a href="<?= $this->Url->build(['controller' => 'EnergyAudits', 'action' => 'view', $energyAudit->id], ['fullBase' => true]) ?>" target="_blank" style="display: inline-block; color: #ffffff; background-color: #3498db; border: solid 2px |
| # |
Sep 10th 2019, 08:18 |
admad |
you are using string `'/favicon-32x32.png'` not url array :slightly_smiling_face: |
| # |
Sep 10th 2019, 08:16 |
challgren |
`<img src=“<?= $this->Url->image(‘/favicon-32x32.png’, [‘fullBase’ => true]) ?>” width=“41" height=“41” alt=“Logo” align=“center” style=“border: none; -ms-interpolation-mode: bicubic; max-width: 100%;“>` |
| # |
Sep 10th 2019, 08:16 |
challgren |
I use it one time |
| # |
Sep 10th 2019, 08:15 |
challgren |
When I first started cake I though it was needed but pretty much stopped using html helper for things like that |
| # |
Sep 10th 2019, 08:15 |
admad |
My opinion is there's no need to :slightly_smiling_face: |
| # |
Sep 10th 2019, 08:14 |
admad |
yup |
| # |
Sep 10th 2019, 08:14 |
challgren |
Like /img/? |
| # |
Sep 10th 2019, 08:14 |
admad |
does any use URL arrays to generate URLs for assets? |
| # |
Sep 10th 2019, 08:14 |
admad |
guys your feedback here would be helpful https://github.com/cakephp/cakephp/pull/13630#issuecomment-529765323 |
| # |
Sep 10th 2019, 08:13 |
challgren |
Yeah… Everyone must be wrapping up projects |
| # |
Sep 10th 2019, 07:55 |
alexdd55976 |
its quiet week in here |
| # |
Sep 10th 2019, 07:07 |
alexdd55976 |
morning fellows |
| # |
Sep 10th 2019, 06:20 |
javier.villanueva |
morning all |