OblikStudio / kirby-link-field

Links field for Kirby.
MIT License
77 stars 10 forks source link

link type names are missing for 5.2.0 (Kirby 3.8.3 / Safari, Chrome / MacOS) #72

Open agloeckner opened 1 year ago

agloeckner commented 1 year ago
Screenshot 2022-12-15 at 10 39 51
hdodov commented 1 year ago

I can't seem to reproduce it. Perhaps there's a conflict with the translation keys, i.e. those things. Have you set up any custom language variables on your site?

agloeckner commented 1 year ago

I am pretty sure it's my Kirby index.php since I revert everything to normal and then the field is looking normal too. I removed all other plugins before too, with still the same error.

<?php

include 'kirby/bootstrap.php';

$kirby = new Kirby([
    'roots' => [
        'accounts' => 'accounts',
        'config' => 'config',
        'languages' => 'languages',
        'assets' => 'app/assets',
        'site' => 'app/site',
        'vendor' => 'app/site',
    ],
]);

echo $kirby->render();
hdodov commented 1 year ago

By changing the roots, perhaps you change the order in which configuration files are loaded (internal Kirby config, plugin config, site/config). If that's the issue, I believe it's out of the scope of this plugin.

agloeckner commented 1 year ago

changed the configuration back to default, same issue, something is wrong with my multi lang setup (en default / de).

when I remove en.php from language folder, the link field still requires en.php:

Screenshot 2023-01-17 at 11 09 40

when I then remove the page.en.txt (only keep page.de.txt) all is fine (with language folder in the root):

Screenshot 2023-01-17 at 11 11 13

agloeckner commented 1 year ago

By changing the roots, perhaps you change the order in which configuration files are loaded (internal Kirby config, plugin config, site/config). If that's the issue, I believe it's out of the scope of this plugin.

understand 👍🏻

hdodov commented 1 year ago

If I were you, I'd set up Xdebug and go line by line to see what Kirby does. It loads configuration differently, by the looks of it. I have had issues before that were caused by file naming/order. Maybe it's the same thing here.