Skip to content
Grav 2.0 is officially stable. Read the announcement →

Community guidelines

Please keep discussions civil and on-topic. Repeated violations may lead to a temporary ban.

Plugins

Asset collections in system.yaml bug?

Started by Bart Uitermark 9 years ago · 3 replies · 771 views
9 years ago

In system.yaml i've added

collections:
uikit:

  • 'system://assets/uikit/uikit.min.css'
  • 'system://assets/uikit/uikit.min.js'
  • 'system://assets/uikit/uikit-icons.min.js'

This works correct.

BUT if i change a random setting through the admin panel and save it, the collecion has changed and it looks like

collections:
'uikit[0': 'system://assets/uikit/uikit.min.css'
'uikit[1': 'system://assets/uikit/uikit.min.js'
'uikit[2': 'system://assets/uikit/uikit-icons.min.js'

and this brakes the template/site...

Am i doing something stupid/wrong or is this some kind of a bug ?

9 years ago

This looks very much like a bug, could you file an issue at the Admin-plugin repo?

9 years ago

I just wanted to be sure, so i've just downloaded the Grav+Admin package installed it clean on my webserver.
Created a user, logged in, cleared the cache, checked for updates, and only add the

uikit:

  • ‘system://assets/uikit/uikit.min.css’
  • ‘system://assets/uikit/uikit.min.js’
  • ‘system://assets/uikit/uikit-icons.min.js’
    under collections: in my system.yaml

didn't even bother to place the uikit files or anything else/extra in the correct location.

Went to the configuration/assets page in the admin plugin, and see the added uikit assets presented as
uikit[0] system://assets/uikit/uikit.min.css
uikit[1] system://assets/uikit/uikit.min.js
uikit[2] system://assets/uikit/uikit-icons.min.js

Just pressed save, and this again changed to
uikit[0 system://assets/uikit/uikit.min.css
uikit[1 system://assets/uikit/uikit.min.js
uikit[2 system://assets/uikit/uikit-icons.min.js

So i'm convinced it's a small but for me anoying bug.
I'll try to file this issue like you suggested.

Thanks.

9 years ago

Confirmed its a bug, and requires a special field that is flexible enough to handle nested elements. I've replied on the github issue.

Suggested topics

Topic Participants Replies Views Activity
Plugins · by Rene, 1 week ago
2 43 1 week ago
Plugins · by Xavier, 4 weeks ago
2 54 4 weeks ago
Plugins · by Luka Prinčič, 7 years ago
3 1181 1 month ago
Plugins · by Sebastian van de Meer, 1 month ago
1 48 1 month ago
Plugins · by PIERROT Alain, 2 months ago
3 73 2 months ago