Project

General

Profile

Bug #3538

Billetkøbeknapper til Place2Book og titelbilleder mangler på alle arrangementer

Added by Gitte Barlach almost 2 years ago. Updated almost 2 years ago.

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

Description

Efter opgradering til DDB CMS release 28 mangler alle billet-købeknapperne til Place2Book-arrangementerne.

 

Et work around er at åbne pågældende arrangement til redigering, lave en ubetydelig rettelse og gemme arrangementer. Det får i de fleste tilfælde købe-knappen tilbage.

Der er dog flere arrangementer hvor det ikke lader sig gøre:

a) eksempel på ingen billetkøbe-knap uanset hvad man gør:
https://www.aakb.dk/arrangementer/tendenser-og-debat/den-gode-snak-hvorfor-er-den-saa-vigtig
https://www.aakb.dk/arrangementer/musik-og-film/virkelighedens-game-of-thrones
https://www.aakb.dk/arrangementer/musik-og-film/klassisk-lytteklub-herbert-von-karajan (det er et gratisk arrangement, men der er krav om tilmelding). 
https://www.aakb.dk/arrangementer/tendenser-og-debat/startmoede-for-ivaerksaettere-3

b) Et aflyst arrangement, der skal have fjernet billetkøbeknappen: https://www.aakb.dk/arrangementer/boern/aflyst-dramaloerdag-med-filuren
Men dette lader sig heller ikke gøre

c) arrangementer der er passive, dvs. arrangementet ligger i P2B, men der skal netop ikke vises en billetknap, får allligevel vist billetknappen

Skærmbillede 2018-05-22 18.27.03.png (85.1 KB) Skærmbillede 2018-05-22 18.27.03.png der vises desværre ingen billetknap, men i stedet står der "nothing" Gitte Barlach, 05/22/2018 06:34 PM
arr_uden_bib.PNG (49.1 KB) arr_uden_bib.PNG Marianne Sivertsen, 05/25/2018 03:26 PM

Related issues

Related to DDB CMS - Bug #3568: Manglende titelbilleder ved arrangementer efter opdatering til release 28Closed
Related to DDB CMS - Bug #3608: Arrangementer som ikke kræver billetsalg får "nothing" i billetknapsfeltetClosed
Related to DDB CMS - Bug #3704: Arrangementer mister feltindholdResolved (tag version)
Has duplicate DDB CMS - Bug #3579: Manglende billetknap på betalingsarrangementerClosed

History

#1 Updated by Christel Krabbenhøft almost 2 years ago

  • Assignee set to Rasmus Høymann Laursen

Hej Rasmus! Kan I kigge på denne med det samme?

Releasen er allerede frigivet til drift, så den er kritisk!

#2 Updated by Tue Gaston almost 2 years ago

Titelbilledet er også forsvundet i Ballerup.
Det gælder kun arrangementer.

Det er der stadig i  backenden - men det vises ikke i frontend.

Produktionssite (ding2-7.x-4.2.1): https://bib.ballerup.dk/arrangementer/foredrag-debat/baeredygtig-debatmiddag - arrangementet har titelbillede og billet-købs-knap

Stagingsite (ding2-7.x-4.3.1-rc3): https://ballerup-stg.ddbcms.dk/arrangementer/foredrag-debat/baeredygtig-debatmiddag  - arrangementet mangler titelbillede og billet-købs-knap

#3 Updated by Rasmus Høymann Laursen almost 2 years ago

  • Assignee changed from Rasmus Høymann Laursen to Kristian Pedersen

#4 Updated by Gitte Barlach almost 2 years ago

  • Description updated (diff)

#5 Updated by Gitte Barlach almost 2 years ago

Læg for øvrigt mærke til #3477. Undrer mig lidt over hvilken sag der har forårsaget den type ændringer?

#6 Updated by Gitte Barlach almost 2 years ago

  • Description updated (diff)

#7 Updated by Kristian Pedersen almost 2 years ago

  • Status changed from Needs analysis to Need more info
  • Assignee changed from Kristian Pedersen to Gitte Barlach

De manglende billeder og højst sandsynligt også de manglende billet-knapper skyldes den bug, som er løst med en patch i dette issue https://platform.dandigbib.org/issues/3466

Det der beskrives i #3477, kan ligeledes skyldes denne bug. Features modulet er blevet opdateret på et tidspunkt (af nogle andre end B14), hvilket har slettet end en del gruppering af felter i backenden og også kan have rykket rundt på felter.

For at kunne teste fjernelse af billetknap og passive arrangementer skal jeg bruge et admin log-in til aakb, hvor problemet eksisterer.

#8 Updated by Jesper Kristensen almost 2 years ago

Efter at have gravet i det tror jeg det er gået i ged her: https://github.com/ding2/ding2/commit/e0a020783fd345ac8527a86cdef9f1f194a049d7

Altså i https://github.com/ding2/ding2/pull/1010

Som jeg ser det er det hele lavet om til at feltet til tænk "external ticket link" bliver udnytte til at gemme link til place2book osse og når der ikke er noget renderes det ikke, men der er ingen update functions til at flytte data.

Det hele ser forkert ud.

----------------------------------

Trick til at få en book knap:
1) Edit event og indsat false link i “Book ticket/Participate in event” felt
2) Gem event
3) Edit event - slet ALT i “Book ticket/Participate in event” felt
4) Gem event

Nu er det på magisk vis kopieret link til p2b eventet ind i “Book ticket/Participate in event” og nu har den en knap… dog ikke en knap det tager højde for salgs status

----------------------------------

Dette trick kan selvfølgelig ikke anbefaldes, men kan være en nød-løsning.

#9 Updated by Gitte Barlach almost 2 years ago

  • Status changed from Need more info to Ready for development
  • Assignee changed from Gitte Barlach to Martin Cording

Martin,  kan I se på dette nu med det samme?  Vi har desværre fejlen ude i produktion ved bibliotekerne. 

#10 Updated by Gitte Barlach almost 2 years ago

Vil gerne tilføje: grundlæggende forstår jeg ikke hvorfor P2B linket skal flyttes til external-feltet, og heller ikke hvorfor vi ikke skulle vise status fra P2B på knapperne (udsolgt, venteliste etc.). Derfor tænker jeg der er tale om en fejl. 

#11 Updated by Martin Cording almost 2 years ago

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

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

Ældre begivenheder fik ikke feltet opdateret som bruges til generering af Book-knappen, så nu bruges den eksisterende funktion der kaldes ved node visning og hvis feltet er tomt, laves en anmodning om at P2B og svaret vil blive brugt med rigtig status.

#12 Updated by Gitte Barlach almost 2 years ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#13 Updated by Jesper Kristensen almost 2 years ago

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

Mindere manglede documentaion og så driller CI igen.

#14 Updated by Martin Cording almost 2 years ago

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

Fixed

#15 Updated by Gitte Barlach almost 2 years ago

hej Martin 

 

Kan du oplyse her hvad jeg skal teste, dvs. hvad er det helt konkret I har lavet med Jeres seneste rettelse ?

#16 Updated by Jesper Kristensen almost 2 years ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jesper Kristensen to Gitte Barlach

Reviewed og afventer release

#17 Updated by Gitte Barlach almost 2 years ago

  • Status changed from Reviewed to Reviewed - Needs info/rework
  • Assignee changed from Gitte Barlach to Martin Cording

Har testet denne på upgrade-fbs. 

 

a) der er nu P2B links i feltet "Book ticket/Participate in event". MEN der vises stadig ingen billetknap, der står bare "nothing". Se vedlagte eksesmpel eller se denen url:https://upgrade-fbs.ddbcms.dk/arrangementer/udstilling/classic-car-race
Har I et bud på hvorfor ? 

b) kan I bekræfte at de billetknapper der vises i "Book ticket/Participate in event" (når de altså engang vises) bliver knapper uden P2B status, dvs. der kommer ikke til at stå "udslogt" eller "venteliste" på knapperne?

#19 Updated by Rolf Madsen almost 2 years ago

  • Related to Bug #3568: Manglende titelbilleder ved arrangementer efter opdatering til release 28 added

#20 Updated by Rolf Madsen almost 2 years ago

  • Subject changed from Billetkøbeknapper til Place2Book mangler på alle arrangementer to Billetkøbeknapper til Place2Book og titelbilleder mangler på alle arrangementer

#21 Updated by Martin Cording almost 2 years ago

  • Status changed from Reviewed - Needs info/rework to Need more info
  • Assignee changed from Martin Cording to Gitte Barlach

a) Logikken omkring hvad der skal vises er ikke ændret, og det ser ud som følger:

switch ($type) {
  case 'event-over':
    print '<div class="place2book-tickets-label">'. t('Tickets') . '</div><div class="place2book-tickets-info">' . t('The event has already taken place') . '</div>';
    break;
  case 'closed-admission':
    print '<div class="place2book-tickets-label">'. t('Tickets') . '</div><div class="place2book-tickets-info">' . t('Not open for ticket sale') . '</div>';
    break;
  case 'no-tickets-left':
    print '<div class="place2book-tickets-label">'. t('Tickets') . '</div><div class="place2book-tickets-info">' . t('Sold out') . '</div>';
    break;
  case 'order-link':
    print l(t('Book a ticket'), $url, array('attributes' => array('class' => array('ticket-available', 'button'))));
    break;
  case 'sale-not-started':
    print '<div class="place2book-tickets-label">'. t('Tickets') . '</div><div class="place2book-tickets-info">' . t('Ticket sale has not yet started for this event') . '</div>';
    break;
  default:
    print '-nothing-';
    break;
}

Det vil altså sige at lige netop dette event ikke har nogen status - hvorfor ved jeg ikke.
Vores kode ændrer i øvrigt dette så den outputter '' (ingenting) og ikke '-nothing-'.

 

b) Ikke helt sikker på jeg forstår dette spørgsmål - hvis der er en status fra P2B (som defineret ovenfor), så bliver de vist på denne knap - ligesom "-nothing-" i dette eksempel.

#22 Updated by Gitte Barlach almost 2 years ago

vedr. a) ok - det lyder jo godt. Men - Af en eller anden grund er der *ingen* af test-arrangementerne på upgrade-fbs der har en billetknap. De står alle med "Nothing" - se https://upgrade-fbs.ddbcms.dk/arrangementer.&nbsp;
vedr. b) det er korekt forstået at jeg spurgte ind til om billet-status stadig ville fremgå af knappen - det er fint den gør det. 

#23 Updated by Gitte Barlach almost 2 years ago

vedr. a) jeg bedt Place2Book kigge på vores testdata i deres ende samt forholde sig til den status der giver "nothing"

#24 Updated by Martin Cording almost 2 years ago

  • Status changed from Need more info to Needs code review

https://github.com/ding2/ding2/pull/1104/files

Med dette PR hentes URL til p2b for hver page load og erstatter tom felt med link og status.
Dette skulle afhjælpe at der skrives "-nothing-".

#25 Updated by Gitte Barlach almost 2 years ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

 @Kasper: denne kom desværre for sent til at komme med i den nye RC, du er i gang med at bygge. Men kan se på det alligevel her i dag ?

#26 Updated by Marianne Sivertsen almost 2 years ago

Når man opretter passive begivenheder, får de alligevel en 'Køb billetter'-knap. Jeg er lidt i tvivl, om det er noget af dette, I allerede har behandlet i denne tråd?

I må bare sige til, hvis jeg skal oprette et nyt issue på det.

#27 Updated by Marianne Sivertsen almost 2 years ago

Der mangler også tilknyttet bibliotek på begivenhederne.

#28 Updated by Gitte Barlach almost 2 years ago

Vedr. den manglende visning af sted/bibliotek så skyldes det denne fejll som er rettet og godkendt, men som endnu ikke er rullet ud: #3466

#29 Updated by Marianne Sivertsen almost 2 years ago

Her er et på et arrangement uden tilknyttet bibliotek. Dette er oprettet inden update 28.
Det er for øvrigt kun inde på arrangementet, at biblioteket mangler - i listevisningen står det.

Mht billet - så har jeg lige slettet de arrangementer jeg havde siddet og oprettet uden at bemærke, at der kom billetknap på, selv om jeg havde sat det som passivt arrangement.
Det er et klonet arrangement, skal jeg lige nævne.

#30 Updated by Kasper Garnæs almost 2 years ago

  • Status changed from Needs code review to Technical test
  • Assignee changed from Kasper Garnæs to Gitte Barlach

Godkendt og merged.

#31 Updated by Tue Gaston almost 2 years ago

Testet og godkendt.

#32 Updated by Tue Gaston almost 2 years ago

  • Status changed from Technical test to Resolved (tag version)

#33 Updated by Rolf Madsen almost 2 years ago

  • Related to Bug #3608: Arrangementer som ikke kræver billetsalg får "nothing" i billetknapsfeltet added

#34 Updated by Rolf Madsen almost 2 years ago

  • Has duplicate Bug #3579: Manglende billetknap på betalingsarrangementer added

#35 Updated by Tue Gaston over 1 year ago

  • Related to Bug #3704: Arrangementer mister feltindhold added

Also available in: Atom PDF