Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

array_flip fails with uuid in array #6537

Closed
magicsepp opened this issue Dec 8, 2013 · 11 comments
Closed

array_flip fails with uuid in array #6537

magicsepp opened this issue Dec 8, 2013 · 11 comments

Comments

@magicsepp
Copy link

https://github.com/contao/core/blob/master/system/modules/core/elements/ContentGallery.php#L220

Hallo Leo,
an verschiedenen Stellen wird array_flip mit den neuen uuid (binary) verwendet das führt zu
Warning: array_flip(): Can only flip STRING and INTEGER values! in D:\xampp\htdocs\strehlweb\system\modules\core\elem ents\ContentGallery.php on line 220

@Babelfisch
Copy link

Hängt vermutlich hiermit zusammen: #6534. Beim Update wird immer nur der erste Datensatz der order-Spalte von multi-Feldern richtig konvertiert und alle anderen haben einen undefinierten Zustand, der zu der Warnung führt.

@leofeyer
Copy link
Member

leofeyer commented Dec 9, 2013

Es ist wohl so, wie @Babelfisch sagt, denn binäre UUIDs sind auch Strings und führen bei mir nicht zu dem Fehler.

@leofeyer leofeyer closed this as completed Dec 9, 2013
@ghost
Copy link

ghost commented Feb 14, 2014

Ist es möglich, dass dieser Fehler nach dem Contao Update 3.2.7 wieder aufgetaucht ist.
Warning: array_flip(): Can only flip STRING and INTEGER values! in ...\system\modules\core\elements\ContentGallery.php on line 220
Wir hatten bei einer Installation genau diesen Fehler wieder, Nach dem Downgrade auf 3.1.1 ist er wieder weg...

@leofeyer leofeyer removed this from the 3.2.2 milestone Feb 14, 2014
@fjacobi
Copy link

fjacobi commented Feb 14, 2014

Stammt vom Update und liegt wohl an der individuellen Sortierung.
Es hilft meist das betreffende CE nochmal zu öffnen, neu zu speichern bzw. die Reihenfolge nochmal zu ändern.

Zum Teil etwas nervig, aber funktioniert ;)
Siehe auch:
https://community.contao.org/de/showthread.php?46285-Probleme-mit-Nachrichten-Eweiterung-beim-Update-auf-3-2

@ghost
Copy link

ghost commented Feb 14, 2014

Neu gespeichert - eh voila. Danke für den Tipp! Nicht schön aber pragmatisch.

@geraldo0409
Copy link

beim update auf 3.2.7 kann ich das ebenfalls bestätigen.
Hat man mehrere Galerien in dem selben Artikel muss man meiner Beoabachtung nach alle zuerst auf Keine Sortirung = zufällige Reihenfolge setzen.
Bei Mehfacher Selektion ist es relativ zufällig, wieviel Galerien man GLEICHZEITIG wieder auf "indivduelle" Sortierung setzt

@takkk
Copy link

takkk commented Feb 18, 2014

kann den bug bestätigen. version 3.2.7

@aschempp
Copy link
Member

aschempp commented Mar 5, 2014

Just had this issue too...

@geraldo0409
Copy link

Hallo andreas
Eine beobachtumg, die evetull hinweis gibt was buggy ist:
2-x galeriem hintereinamder.
Multiedit sort zufall : alles ok!
Dann multiediedit alle indiv FEHLER BLEIBT
Jede galerie Einel Ok

Sorry smartphone tastatir ein graus

@BugBuster1701
Copy link
Contributor

Bestätigung: Bei Update von 3.1.5 auf 3.2.7 bei MA Demo in Seite "Impressions"

@leofeyer leofeyer added this to the 3.2.8 milestone Mar 8, 2014
@leofeyer leofeyer reopened this Mar 8, 2014
@leofeyer
Copy link
Member

Tut mir leid, aber ich kann das Problem einfach nicht reproduzieren. Hier mein Vorgehen:

  1. Checkout der Version 3.1.5.
  2. DB-Update mit dem Installtool und Import der Music Academy.
  3. Login im Backend und Ändern der Reihenfolge der Bilder auf der Impressions-Seite.
  4. Checkout der Version 3.2.7.
  5. DB-Update mit dem Installtool ohne erneuten Import der Music Academy.
  6. Aufruf der Impressions-Seite im Frontend -> keine Fehlermeldung, Reihenfolge stimmt.

An welchem Punkt weichen eure Ergebnisse ab?

@leofeyer leofeyer removed this from the 3.2.8 milestone Mar 12, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants