Project

General

Profile

Enhancement #4394

Place2book: Kloning af arrangement opretter ikke ny billet i place2book

Added by Stefan Søndervang 4 months ago. Updated about 1 month ago.

Status:
Resolved (tag version)
Priority:
Urgent
Assignee:
Target version:
Estimated time:
URL med eksempel:
Kategorier:
Inspiration - Arrangementer, Integration - Place2book (Arangementer)

Description

Problem

Når man kloner et arrangement, der har en place2book billet tilknyttet, så opretter den ikke en ny billet. I stedet redigeres den billet, der er til det arrangement man oprindeligt klonede. Det betyder at begge arrangementer, som skal have to forskellige billetter får den samme billet tilknyttet.

Eksempel case:

  • Jeg har et arrangement, som finder sted to gange på to forskellige steder og tidspunkter. Jeg ønsker at det skal være Arrangement 1 med Billet 1 og Arrangement 2 med Billet 2.
  • Jeg opretter det første arrangementer (Arrangement 1) og opretter en place2book billet via place2book fanen (Billet 1). 
  • Jeg går ind i /admin/content og kloner arrangementet.
  • Derved oprettes Arrangement 2.
  • Jeg ændrer titel, tid og sted og trykker Gem.
  • I stedet for at jeg får Billet 2, så har jeg ændret Billet 1, så både Arrangement 1 og Arrangement 2 er linket til Billet 1.
  • Jeg har tjekket om jeg kan klikke Vedligholdelse i place2book fra og til for at oprette en ny, men der sker ingen forskel.

Løsningsforslag

Når der klones et arrangement, så skal den billet der er oprettet i det oprindelige arrangement ikke følges med til det klonede arrangement.

 


Related issues

Is duplicate of DDB CMS - Enhancement #1663: Tilføje mulighed for at oprette en serie af gentagne arrangementerReviewed - Needs info/rework
Has duplicate DDB CMS - Bug #4409: Klonede arrangementer overskiver originalen i P2B arrangøroversigtClosed

History

#1 Updated by Martin Cording 4 months ago

  • Status changed from Ready for development to Needs code review
  • Assignee changed from Martin Cording to Gitte Barlach

#2 Updated by Gitte Barlach 4 months ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen

#3 Updated by Fanan Imad 4 months ago

Vi har prøvet at klone og fjerne den eksisterende billet og oprette en ny, det volder dog desværre stadig problemer! Når man f.eks. tilmelder sig et oprindeligt klonet arrangement, der fx finder sted i december, lægger tilmeldingen sig under under det 'oprindelige' arrangement/moder-arrangementet i september.

#4 Updated by Rolf Madsen 4 months ago

  • Is duplicate of Enhancement #1663: Tilføje mulighed for at oprette en serie af gentagne arrangementer added

#5 Updated by Rolf Madsen 4 months ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Jørgen Nielsen to Martin Cording

Jeg mener dette er et duplikat af #1663.

@Martin vil du beskrive hvad din løsning gør?

#6 Updated by Martin Cording 4 months ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Martin Cording to Rolf Madsen

#1663 handler om gentagne arragnementer og har ikke noget med dette problem at gøre.

Vores løsning betragter kloning som en "ny" node og fjerner relationen til Place2book API'et.

#7 Updated by Gitte Barlach 4 months ago

  • Assignee changed from Rolf Madsen to Kasper Garnæs

#8 Updated by Kasper Garnæs 4 months ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Kasper Garnæs to Martin Cording

Reviewd. Jeg har en kommentar til hvordan dette er løst.

#9 Updated by Simon Holt 4 months ago

Den får vel også det samme event i p2b tilknyttet. Er det virkelig det vi vil have? Hvordan skal man f.eks. få det klonede arrangement til at pege på et nyt arrangement i p2b uden at slette det oprindelige?

Måske er det bedre at sørge for, at alt p2b fjernes fra det klonede arrangement. Således skal man selv tilvælge p2b på redigeringssiden for det klonede arrangement og når man trykker gem vil den så korrekt oprette et nyt arrangement i p2b.

 

#10 Updated by Christel Krabbenhøft 4 months ago

Martin, vil du tilpasse ud fra Simons kommentar "Måske er det bedre at sørge for, at alt p2b fjernes fra det klonede arrangement."?

#11 Updated by Martin Cording 4 months ago

  • Assignee changed from Martin Cording to Kasper Garnæs

Simon: Det er vel netop det der gøres i denne commit? :)

Kasper: Der er svaret på din kommentar - det er ikke rigtig muligt. Der er foreslået alternativ løsning.

#12 Updated by Simon Holt 3 months ago

Simon: Det er vel netop det der gøres i denne commit? :)

Nej. I sørger for at billettyperne ikke anvender de samme ticket_ids, men I rører jo ikke ved event_id, så arrangmentet peger stadig på det samme event i p2b som originalen gør.

Så hvis man trykker gem, vil der bare blive oprettet nye billettyper på det samme event i p2b.

Det dur ikke det er det samme event i p2b begge arrangementer peger på, da tidspunktet for event skal være anderledes i klonen (ellers er der ingen grund til at klone - så kan man lige så godt bruge det samme arrangementet). Formålet med at klone arrangementer er, at man har et identisk arrangement der skal gentages på flere forskellige tidspunkter.

#13 Updated by Simon Holt 3 months ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Kasper Garnæs to Gitte Barlach

Vi står og skal bruge denne funktionalitet nu, så har tilladt mig at fremsætte et nyt PR, der sørger for at klonet arrangement ikke peger på samme i p2b. Synes også det er løst på en bedre måde med hook_node_clone_alter().

https://github.com/ding2/ding2/pull/1488

#14 Updated by Simon Holt 3 months ago

Nej. I sørger for at billettyperne ikke anvender de samme ticket_ids, men I rører jo ikke ved event_id, så arrangmentet peger stadig på det samme event i p2b som originalen gør.

Det beklager jeg, Martin. ret skal være ret: kan godt se i sætter is_new og dermed opretter ny event. Det jeg blev snydt af var, at den ikke opdaterer sales URL i ticket link feltet, hvilket er oplagt at gøre fra node_clone_alter. Der er også et andet problem når man kloner flere noder via dropdown i indholdsoversigt. Har skrevet om det her

#15 Updated by Rolf Madsen 3 months ago

  • Has duplicate Bug #4409: Klonede arrangementer overskiver originalen i P2B arrangøroversigt added

#16 Updated by Gitte Barlach 3 months ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

#17 Updated by Kasper Garnæs 2 months ago

  • Status changed from Needs code review to Reviewed

Ændringen i https://github.com/ding2/ding2/pull/1488 er reviewed og godkendt.

#18 Updated by Kasper Garnæs 2 months ago

  • Status changed from Reviewed to Technical test

#19 Updated by Kasper Garnæs 2 months ago

  • Assignee changed from Kasper Garnæs to Gitte Barlach

#20 Updated by Stefan Søndervang about 2 months ago

  • Assignee changed from Gitte Barlach to Stefan Søndervang

#21 Updated by Stefan Søndervang about 2 months ago

  • Status changed from Technical test to Reviewed - Needs info/rework
  • Assignee changed from Stefan Søndervang to Simon Holt

@Simon jeg har testet på ding2-7.x-4.8.0-rc5 og får nedenstående problem.

Testens fremgangsmåde

  1. Jeg har oprettet et arrangement og opretter en billet i place2book
  2. Jeg går ind og kloner arrangementet her: /admin/content
  3. Jeg ændrer tidpunktet fra 8.50 til 12.50 i klonen.
  4. Jeg folder Place2book fanen ud, og kan se at det er den gamle billet der er tilknyttet.
  5. Jeg antager at de fleste webfolk ikke vil ligge mærke til dette og går videre i testen uden at rette noget. Så jeg gemmer bare arrangementet.
  6. Jeg får nedenstående fejl.
  7. Der er ikke en billet tilknyttet arrangementet.
  8. Jeg redigerer arrangementet igen og udfolder Place2book fanen, og så vil den gerne oprette den korrekte billet.

Ved punkt 4-6 skal den gamle billet ikke være tilknyttet.

#22 Updated by Simon Holt about 2 months ago

Hmm.. underligt. Tager lige et kig på det.

Mht. 4-6. Lige for at være sikker på vi er enige: Det skal ikke være den gamle billet tilknyttet, men data fra billetttyperne fra det gamle arrangment skal vel overføres og indsættes i formen? Det er jo sådan det fungerer med de andre klonede felter. F.eks. hvis man har oprettet et arrangement med tre billettyper og man kloner dette, så skal formularen sættes op med de "samme" billettyper, men når man trykker gem er det selvfølgelig nye billettyper der oprettes i p2b og associeret med det nye arrangement.

#23 Updated by Stefan Søndervang about 2 months ago

Jeg tror det er rigtigt at billettyperne skal klones. Så længe arrangementsoplysningerne fx tid, sted ikke tages med fra det klonede arrangement, men hentes fra det arrangement som man er i gang med.

#24 Updated by Simon Holt about 2 months ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Simon Holt to Gitte Barlach

@Stefan har nu testet og fik samme fejl. Fejlen skyldes egentlig rettelserne i https://platform.dandigbib.org/issues/4446, men lad os bare tage den her.

Nyt PR: https://github.com/ding2/ding2/pull/1514

#25 Updated by Gitte Barlach about 2 months ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

#26 Updated by Kasper Garnæs about 2 months ago

  • Status changed from Needs code review to Reviewed

Reviewed og godkendt.

#27 Updated by Kasper Garnæs about 1 month ago

  • Status changed from Reviewed to Technical test

Merged.

#28 Updated by Kasper Garnæs about 1 month ago

  • Assignee changed from Kasper Garnæs to Gitte Barlach

#29 Updated by Stefan Søndervang about 1 month ago

  • Assignee changed from Gitte Barlach to Stefan Søndervang

#30 Updated by Stefan Søndervang about 1 month ago

  • Status changed from Technical test to Resolved (tag version)
  • Assignee changed from Stefan Søndervang to Gitte Barlach

Testet og godkendt på ding2-7.x-5.0.0-beta3.

Testens fremgangsmåde

  1. Jeg har oprettet et arrangement og opretter en billet i place2book
  2. Jeg går ind og kloner arrangementet her: /admin/content
  3. Jeg ændrer tidpunktet fra 8.50 til 12.50 i klonen.
  4. Jeg folder Place2book fanen ud, og kan se at det er den gamle billet der er tilknyttet.
  5. Jeg trykker Gem.
  6. Begge arrangementer har den rigtige billet.

Det er lidt forvirrende at det er den gamle billet der er tilknyttet i punkt 4. Men det virker nu. Hvis forvirringen er for stor må det oprettes som en sag senere.

#31 Updated by Simon Holt about 1 month ago

Det er lidt forvirrende at det er den gamle billet der er tilknyttet i punkt 4. Men det virker nu. Hvis forvirringen er for stor må det oprettes som en sag senere.

Det er ikke den gamle billet. Den kloner billeterne fra det gamle arrangement.

Vil mene det er såden det bør virke og det er også sådan det virker for resten af felterne. Hvis man eksempelvis har oprettet et arrangment med 3 billettyper og man kloner det, så skal det klonede arrangment også have de 3 billettyper, så man ikke skal oprette dem igen på det klonede arrangement. Det er jo netop det, der er ideen med at klone.

Hvis man ikke vil have billettyperne med på det klonede arrangement, skal man klone det før man opretter billettyperne. Man har dermed mulighed for at gøre det på begge måder. I modsætning til hvis vi altid bare fjernede billettyperne, ville man altid være tvunget til at skulle genoprette dem på det klonede arrangement.

#32 Updated by Stefan Søndervang about 1 month ago

Det giver jeg dig ret i. Jeg tror at jeg udtrykte mig lidt for dovent.

Problemet er at hvis jeg har et arrangement som skal klones. Det oprindelige arrangement A har Titel A, Tidspunkt A og Sted A. Men det nye arrangement B har Titel B, Tidspunkt B og Sted B. Hvis jeg kloner arrangement A til arrangement B, og åbner place2book fanen så er informationerne Titel A, Tidspunkt A og Sted A. Også selvom jeg har rettet oplysninger i arrangementet til Titel B, Tidspunkt B og Sted B. Så det virker som om jeg opretter en billet med det gamle tidspunkt, sted eller titel.

#33 Updated by Simon Holt about 1 month ago

Tak for uddybning, Stefan!

Kan godt se din bekymring nu. Men indstillingerne for start og slut dato er jo også kopieret fra det originale arrangement, så vil mene det er det samme. Det er jo selvfølgelig den forskel, at man ikke kan ændre på teksten, der viser de originale datoer. Så man skal indse, at disse også vil blive ændret efter man har trykket gem, hvis man ændret datoerne for arrangementet længere oppe. Det er måske ikke så indlysende. Vi kunne fjerne disse datoer fra det klonede p2b-arrangement, så de ikke bliver vist i teksten. Men det vil måske give anledning til anden forvirring.

Jeg tror det bedste er at beholde det som det er nu og hvis man vil gentage et arrangement flere gange over tid, er det nok mere funktionalitet til "Gentagelse af arrangementer" man skal have fat i, som forhåbentlig snart har klar i: https://platform.dandigbib.org/issues/1663

#34 Updated by Stefan Søndervang about 1 month ago

Det er også derfor jeg godkendte sagen. Tror det er et ikke-problem. Men kan godt lide at skrive mine bekymringer, så man kan gå tilbage og læse dem. 

Also available in: Atom PDF