search

TN03 / expandcontract

http://svasti.de/en/?Welcome/Expandcontract
0 stars 1 forks source link

Doppelte IDs verhindern #3

Closed TN03 closed 2 years ago

TN03 commented 2 years ago

In zwei gleich konfigurierten Instanzen dürfen identische Seiten nicht integriert werden, weil sonst Ids doppelt vergeben werden. Bei unterschiedlich konfigurierten Instanzen ist das zwar möglich, aber nur zu Demonstrationszwecken nützlich, siehe https://github.com/TN03/expandcontract/blob/890b939b56d04ca3f19e555a4b681431826ab0f4/index.php#L35-L36

Die Ids werden mittels der (Seiten-) Nummer in $h generiert. Das ist, insbesondere für direkte Verlinkungen ungünstig, da diese Nummer nicht permanent ist und sich zum Beispiel schon beim verschieben einer Seite im Seitenbaum verändert.

Mindestens sollten die Ids für jede Instanz in der Seite unterschiedlich werden.

TN03 commented 2 years ago

Hierzu gibt es jetzt einen PR, siehe: https://github.com/TN03/expandcontract/pull/7 Kann auch mit diesem Branch getestet werden: https://github.com/TN03/expandcontract/tree/Uniqe.Ids Eigentlich sollte es keine CSS- oder JS - Probleme geben....

TN03 commented 2 years ago

Geschlossen mit https://github.com/TN03/expandcontract/commit/0689078a1917e5c35c181a1b6463a8cde5d06c5e