20240818 feat warehouse bin presenter

[bblessmann]

No description provided.

[rebootl]

Sieht gut aus von mir her. 👍

Habe es kurz lokal getestet, hat funktioniert, habe aber nur ein Lager.
Code etc. soweit ich sehe alles ok.

[bblessmann]

Hi Cem,

Sieht gut aus von mir her. 👍

Habe es kurz lokal getestet, hat funktioniert, habe aber nur ein Lager. Code etc. soweit ich sehe alles ok.

danke schonmal.
Ich habe kurz getestet, eine vorhandene Lager/Lagerplatz auswahl zu erstetzen. Dabei sind mir noch ein paar Dinge aufgefallen. Ist also noch "WIP".

Grüße
Bernd

[rebootl]

Ich könnte sonst mal noch versuchen ob ich den unter Einkauf -> Einlagern einbauen kann.

LG

[bblessmann]

Hi,
ja, wenn Du magst.
Probleme, die ich hatte waren kleinere:

• im Part-Controller wird die Lager-Auswahl nur gerendert, wenn auch Lager angelegt sind. Das berücksichtigt der Presenter noch nicht
• der Presenter rendert Lager und Lagerplatz nebeneinander, nicht untereindander. Ist aber wohl nicht so wichtig.
• der Presenter rendert immer eine Lagerplatz-Auswahl, auch wenn noch kein Lager gewählt ist. Die Auswahlliste ist dann leer). Der Part-Controller zeigt die Lagerplatzauswahl in diesem Fall gar nicht an
• es hat schlicht nicht funktionert, bzw. ich hatte die alte Auswahl dringelassen zum Vergleichen. Da kamen sich dann aber wohl die Ids der select-tags in die Quere

Grüße

[rebootl]

Hi

• im Part-Controller wird die Lager-Auswahl nur gerendert, wenn auch Lager angelegt sind. Das berücksichtigt der Presenter noch nicht

Ja okay, wobei das könnte man ja sonst allenfalls auch innerhalb vom part-controller lösen.

• der Presenter rendert Lager und Lagerplatz nebeneinander, nicht untereindander. Ist aber wohl nicht so wichtig.

Dazu ist mir aufgefallen dass wenn ich eine klasse übergebe also zum Beispiel: class='wi-wide'
Diese wird dann auf alle Elemente angewendet, was dazu führt dass die Elemente dann untereinander in der entsprechenden breite kommen.

Ein weiteres Problem ist auch dass die Bezeichnung vom Lagerplatz dann fehlt. im part-controller z.B.: "Standard-Lagerplatz"

Da könnte man vielleicht einfach "Standard-Lager / Standard-Lagerplatz" ins Tabellenfeld schreiben

• der Presenter rendert immer eine Lagerplatz-Auswahl, auch wenn noch kein Lager gewählt ist. Die Auswahlliste ist dann leer). Der Part-Controller zeigt die Lagerplatzauswahl in diesem Fall gar nicht an

Immer rendern finde ich auch ok oder eher besser.

• es hat schlicht nicht funktionert, bzw. ich hatte die alte Auswahl dringelassen zum Vergleichen. Da kamen sich dann aber wohl die Ids der select-tags in die Quere

Ja vermutlich.

Eine weitere Sache ist mir noch aufgefallen: Wenn ich das lager ändere und dann die Seite aktualisiere (refresh), dann bleibt das ausgewählte lager erhalten, die Lagerplätze springen aber wieder zurück zu denen vom ersten lager.
Das stimmt dann nicht mehr überein.

LG

[bblessmann]

Hi,

danke schonmal fürs Testen. Meine Anmerkungen inline ...

• im Part-Controller wird die Lager-Auswahl nur gerendert, wenn auch Lager angelegt sind. Das berücksichtigt der Presenter noch nicht

Ja okay, wobei das könnte man ja sonst allenfalls auch innerhalb vom part-controller lösen.

Ja, das ginge, aber dann braucht man aus dem backend die Info dafür (die Größe von "all_warehouses" oder ein Flag). Für den Anwender des Presenters sollte es so einfach wie möglich sein. Ich denke, ich mache da ein Parameter für (dont_show_if_collection_empty -> naja, evtl. fällt mir eine bessere Bezeichnung ein).

• der Presenter rendert Lager und Lagerplatz nebeneinander, nicht untereindander. Ist aber wohl nicht so wichtig.

Dazu ist mir aufgefallen dass wenn ich eine klasse übergebe also zum Beispiel: class='wi-wide' Diese wird dann auf alle Elemente angewendet, was dazu führt dass die Elemente dann untereinander in der entsprechenden breite kommen.

Genau. Ich hatte das so gemacht, dass alle Attribute, die nicht direkt den Presenter/Verhalten steuern, bzw. nur für wh oder bin sind, an alle Tags weitergereicht werden.
Evtl. sollte ich noch die Möglichkeit eines Prefixes einbauen, also z.B.
class für alle
div_class für das div
wh_class für wh-selection-tag
bin_class für bin-selection-tag
für die ids, names hatte ich das ja schon.

Ein weiteres Problem ist auch dass die Bezeichnung vom Lagerplatz dann fehlt. im part-controller z.B.: "Standard-Lagerplatz"

Da könnte man vielleicht einfach "Standard-Lager / Standard-Lagerplatz" ins Tabellenfeld schreiben

Ja, das ist gut. Fand ich nicht so wichtig. Wenn man das ganze als "Lagerplatz" bezeichnet, sollte das ja auch die Auswahl des Lagers, in dem sich der Lagerplatz befindet, implizieren.

• der Presenter rendert immer eine Lagerplatz-Auswahl, auch wenn noch kein Lager gewählt ist. Die Auswahlliste ist dann leer). Der Part-Controller zeigt die Lagerplatzauswahl in diesem Fall gar nicht an

Immer rendern finde ich auch ok oder eher besser.

Ok.

• es hat schlicht nicht funktionert, bzw. ich hatte die alte Auswahl dringelassen zum Vergleichen. Da kamen sich dann aber wohl die Ids der select-tags in die Quere

Ja vermutlich.

Eine weitere Sache ist mir noch aufgefallen: Wenn ich das lager ändere und dann die Seite aktualisiere (refresh), dann bleibt das ausgewählte lager erhalten, die Lagerplätze springen aber wieder zurück zu denen vom ersten lager. Das stimmt dann nicht mehr überein.

Ah - ok. Dann muss ich beim Laden noch schauen, ob ein Lager selektiert ist.

LG
Ebenso Grüße

[rebootl]

Heiii, Du hast es vielleicht selber auch schon gemerkt da stimmt glaube ich etwas noch nicht, wenn ich jetzt bei part schaue...

Da kommt das name Attribut zweimal vor beim oberen select. Edit: stimmt nicht, ich sehe das eine heisst data-bin-dom-name

Wenn ich einen Lagerplatz auswähle wird der nicht gespeichert oder nicht angezeigt. Es springt immer auf den obersten, wenn ich die Seite aktualisiere.

LG

[bblessmann]

Hi Cem,

Da kommt das name Attribut zweimal vor beim oberen select. Edit: stimmt nicht, ich sehe das eine heisst data-bin-dom-name

ja, genau. Ich muss mir den vom Anwender vergebenen Namen und Id merken, damit ich das Lagerplatz select ansprechen kann.
D.h. den Namen brauche ich gar nicht mehr - ich hatte das Anfangs so gemacht, dass ich das Bin-Select neu generiere und dafür den richtigen Namen setzen musste. Ich habe das dann so geändert, dass ich nur die Options anpasse.
Den data-Teil 'bin-dom-name' schmeisse ich dann noch raus.

Wenn ich einen Lagerplatz auswähle wird der nicht gespeichert oder nicht angezeigt. Es springt immer auf den obersten, wenn ich die Seite aktualisiere.

Mhm - ja. Das ist auch bei der herkömmlichen Lagerauswahl so. Vermute, dass hat mit dem Update der Optionen per js zu tun. Da wüsste ich nicht, wie ich das ändern kann.
Hast Du eine Idee?

Viele Grüße
Bernd

[bblessmann]

Wenn ich einen Lagerplatz auswähle wird der nicht gespeichert oder nicht angezeigt. Es springt immer auf den obersten, wenn ich die Seite aktualisiere.

Ah. Da scheint doch noch was nicht zu stimmen - ich schaue nochmal.