Skip to content
This repository has been archived by the owner on Oct 6, 2022. It is now read-only.

Creates content file in multilanguage setup #22

Open
mullema opened this issue Feb 13, 2019 · 5 comments
Open

Creates content file in multilanguage setup #22

mullema opened this issue Feb 13, 2019 · 5 comments
Assignees
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@mullema
Copy link

mullema commented Feb 13, 2019

Reference is this Thread

Multilanguage Setup (e.g. DE (default), EN)

  • Creating a page via the frontend (e.g. $page->createChild($mycontent)) while EN is current language.
  • Kirby creates new pages in default language only. Therefore the content file content.de.txt is created.
  • autoid kicks in via hook and adds the hidden field for the current language thereby creating content.en.txt (because EN is current language)

To create the content.en.txt file at that point is an unexpected side effect from the autoid plugin.

A solution could be to update only existing content files.

@mullema mullema changed the title Creates unwanted content file in multilanguage setup Creates content file in multilanguage setup Feb 13, 2019
@bnomei bnomei self-assigned this Feb 13, 2019
@bnomei bnomei added the bug Something isn't working label Feb 13, 2019
@carstengrimm
Copy link

is there a suggestion for a temporary fix? i would love testing it out with my setup.

@scsskid
Copy link

scsskid commented Nov 7, 2019

is this fixed?

@bnomei
Copy link
Owner

bnomei commented Nov 7, 2019

@scsskid thanks for reminding me again. i still have no clear idea how to achieve the suggested solution to update only existing content files in actual code.

checking if the file exist seems a bit hacky to me. as far as i can tell there are no bugs cause by having the content.en.txt file being created, right?

i am currently working on a new version of the plugin with better multilang setup. i will include this issue in testscases to solve it properly.

@scsskid
Copy link

scsskid commented Nov 7, 2019

I dont use your plugin currently, I just eval the issue list, since looking for sth to replace my legacy kirby2 solution, acutally I furthermore have missunderstood the issue

@bnomei
Copy link
Owner

bnomei commented Jun 14, 2022

might be related to an thus fixed in upcoming kirby 3.7

getkirby/kirby#2789

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants