Project

General

Profile

Bug #3704

Arrangementer mister feltindhold

Added by Christian Vandel over 1 year ago. Updated 11 months ago.

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

Description

Vi har haft henvendelser fra henholdsvis Esbjerg og Hillerød om arrangementer, som mister dele af indholdet. Begge sites har mange loglinier som den her:

190.2.146.73|http://www.esbjergbibliotek.dk/|https://www.esbjergbibliotek.dk/|0||PDOException: SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'node-2564-0-0-und' for key 'PRIMARY': INSERT INTO {field_data_field_ding_event_body} (entity_type, entity_id, revision_id, bundle, delta, language, field_ding_event_body_value, field_ding_event_body_format) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2, :db_insert_placeholder_3, :db_insert_placeholder_4, :db_insert_placeholder_5, :db_insert_placeholder_6, :db_insert_placeholder_7); Array#012(#012    [:db_insert_placeholder_0] => node#012    [:db_insert_placeholder_1] => 2564#012    [:db_insert_placeholder_2] => 2641#012    [:db_insert_placeholder_3] => ding_event#012    [:db_insert_placeholder_4] => 0#012    [:db_insert_placeholder_5] => und#012    [:db_insert_placeholder_6] => <p>Da Leonoras mor dør, beslutter hun at skrive den faktiske historie om sig selv og sit liv. Om at vokse op som den ensomme Christina i en familie, hvor mor endelig ikke må græde, og far endelig ikke må blive vred, og om at flytte til København og genskabe sig selv som forfatteren Leonora, der til sine forældres rædsel forelsker sig i en kvindelig præst. ”Når du ligger i rendestenen, skal du nok komme hjem”, siger hendes far, men så tager Leonoras raseri og livslyst over.</p><p>I foredraget fortæller Leonora om den voldsomme skriveproces, når man smider fiktionens forklædninger og skriver om faktiske mennesker, om at miste det hele og finde sig selv, og om at håbe på en forklaring og hige efter sin mors kærlighed langt ind i døden. Et levende og engageret foredrag krydret med oplæsning og spørgsmål fra salen.</p><p>Leonora Christina Skov (f. 1976) er aktuel med erindringsromanen, Den, der lever stille, og er desuden forfatter til romanerne Hvor intet bryder vinden (2015), Førsteelskeren (2012), Silhuet af en synder (2010), Champagnepigen (2007) og Rygsvømmeren (2003). Forfatterskabet er solgt til udgivelse i Norge, Spanien, Tyskland, Holland, Tjekkiet og Serbien. Leonora er mangeårig litteraturanmelder ved Weekendavisen, mag.art. i litteraturvidenskab.</p><p>Billetter købes her eller på Bramming Bibliotek.</p><p>Arrangeret i et samarbejde mellem Bramming Bibliotek, Grundtvigsk Højskoleforening, Bramming Menighedsråd og Foreningen Aktive Kvinder i Bramming.</p>#012    [:db_insert_placeholder_7] => ding_wysiwyg#012)#012 in field_sql_storage_field_storage_write() (line 514 of /var/aegir/platforms/esbjerg180716131208/modules/field/modules/field_sql_storage/field_sql_storage.module).

Det virker lidt tilfældigt hvilket field der rammes, men altid fra ding_event.

Spørgsmålet er så hvad det er på forsiden der trigger en opdatering af nodeindholdet.

Mange af logningerne stammer fra en bestemt IP, 190.2.146.73, som jeg mistænker for at være en harvester fra en content farm, men der er også logninger fra "almindelige" IP'er. Det med content-farmen får mig til at fundere på om det kan være relateret til sprog og oversættelse, men det er rent gætværk...

 

ddbcmstest.JPG (90.3 KB) ddbcmstest.JPG Tanja Kate Jensen, 09/04/2018 12:40 PM
test_no_tickets_left.JPG (155 KB) test_no_tickets_left.JPG Tanja Kate Jensen, 09/07/2018 11:24 AM
test_event_over.JPG (169 KB) test_event_over.JPG Tanja Kate Jensen, 09/07/2018 11:24 AM
test_ældste_arrangement.JPG (130 KB) test_ældste_arrangement.JPG Tanja Kate Jensen, 09/07/2018 11:24 AM
Skærmbillede 2018-09-07 11.15.08.png (245 KB) Skærmbillede 2018-09-07 11.15.08.png "tickets" i øverste venstre hjørne Gitte Barlach, 09/07/2018 11:33 AM
udsolgt_arr.PNG (5.18 KB) udsolgt_arr.PNG Carsten Vilhelmsen, 09/28/2018 11:55 AM

Related issues

Related to DDB CMS - Bug #3751: Fejl i arrangementer med fri entre og med billetsalg (også staging) Closed
Related to DDB CMS - Bug #3538: Billetkøbeknapper til Place2Book og titelbilleder mangler på alle arrangementerResolved (tag version)
Related to DDB CMS - Bug #3741: Manglende billetknapper hos Gentofte og Gladsaxe BibliotekerClosed
Related to DDB CMS - Bug #3608: Arrangementer som ikke kræver billetsalg får "nothing" i billetknapsfeltetClosed

History

#1 Updated by Rolf Madsen over 1 year ago

  • Status changed from New to Ready for development
  • Assignee set to Christel Krabbenhøft
  • Target version set to Release 30 - BPI, Kampagneplus og Sektioner (7.x-4.6.0)

#2 Updated by Christel Krabbenhøft over 1 year ago

  • Assignee changed from Christel Krabbenhøft to Martin Cording

Hej Martin. Vil I tage et kig på denne?

#3 Updated by Christian Vandel over 1 year ago

Problemet er nu også opstået på Hvidovres site. Da indhold går tabt må det betragtes som en yderst kritisk fejl, dvs. den bør rettes før release 30!

 

#4 Updated by Christel Krabbenhøft over 1 year ago

  • Target version changed from Release 30 - BPI, Kampagneplus og Sektioner (7.x-4.6.0) to Release 29-2 - Bugfixes (Inlead)

#5 Updated by Gitte Barlach over 1 year ago

Martin har I tid til at se på denne i indeværende uge?

#6 Updated by Christel Krabbenhøft over 1 year ago

Gitte, Martin har allerede skrevet, at de kigger på den i morgen:-)

#7 Updated by Martin Cording over 1 year ago

Er der mere information omkring dette problem?

F.eks. hvordan og hvornår sker det?

#8 Updated by Christian Vandel over 1 year ago

Det ser ud til at ske fra forsiden. Derudover er der ikke rigtig noget mønster - udover at det, som jeg nævner, ser ud som om at en høster fra en content farm har været særlig slem til at udløse det...

#9 Updated by Martin Cording over 1 year ago

  • Assignee changed from Martin Cording to Christian Vandel

Kan dette være et sikkerhedsbrud?

Dét at skrive til field_data_field_ding_event_body fra forsiden virker meget underligt og burde ikke være muligt?

#10 Updated by Rolf Madsen over 1 year ago

Det er indtil videre tre forskellige biblioteker alle tre på Webmasterhostingplanen.

Man kunne forestille sig at de:

  1. Ikke har skiftet password siden de oprindeligt fik tildelt lokal administrator brugeren.
  2. Har delt det samme password imellem en række medarbejdere i stedet for at bruge editor rollerne.

Men ville man have adgang til det felt med en lokal administrator adgang?

#11 Updated by Rolf Madsen over 1 year ago

  • Priority changed from Urgent to Immediate

#12 Updated by Rolf Madsen over 1 year ago

Der er tale om tre webmasterbiblioteker.

#13 Updated by Rolf Madsen over 1 year ago

  • Status changed from Ready for development to Needs analysis
  • Assignee changed from Christian Vandel to Martin Cording

#14 Updated by Christian Vandel over 1 year ago

Det er ikke et bestemt felt, som rammes. Og som sagt ser det ud til at ske i forbindelse med visning af forsiden, så selvom jeg ikke vil udelukke noget, peger det ikke i retning af bevidst manipulation. 

#15 Updated by René Krøll over 1 year ago

Det har nu også ramt Ballerup.

Vi har en del forskellige arrangementer som enten forsvinder helt eller hvor forskellige felter pludselig ikke er udfyldt og som dermed forhindre arrangementet i at blive vist på siden.

 

#16 Updated by Christian Vandel over 1 year ago

Er det korrekt forstået at place2book modulet kan finde på at opdatere event-noden i forbindelse med visning? I så fald bør der nok kigges i den retning...

#17 Updated by Rolf Madsen over 1 year ago

  • Assignee changed from Martin Cording to Christian Vandel

Christian vil du lave et full site dump af en instans hvor det er sket og information omkring hvilke node id det er sket for?

#18 Updated by Rolf Madsen over 1 year ago

#3608 Arrangementer som ikke kræver billetsalg får "nothing" i billetknapsfeltet - https://platform.dandigbib.org/issues/3608#note-12 har været til Code review hvis den kan være en del af løsningen. Kræver dog rebase og mulig tilpasning til seneste release.

#19 Updated by Jesper Kristensen over 1 year ago

@rolf jeg tror ikke på at #3608 har noget at gøre med dette og om vendt.

Enest løsning er at få et dump at både kodebase og database (altså et drush archive) af et af de sites der fejler og beskrivelse af hvad der fejler også få en "ddb GDPR godkendt" udvikler til at kigge på det.

#20 Updated by Rolf Madsen over 1 year ago

  • Assignee changed from Christian Vandel to Martin Cording

@Martin, så du have oplysningerne til at hente dumpet i din indbakke.

#21 Updated by Rolf Madsen over 1 year ago

Jeg har spurgt tre af de ramte biblioteker om om de før de opdagede at der var forsvundet indhold:

  • Opdateret DDB CMS

  • installeret nye 3. partsmoduler og hvilke var det i så fald

  • Redigéret i de begivenheder hvor indholdet er forsvundet fra

Hillerød skriver følgende:

 

Vi opdaterede CMS’en tidligere på sommeren i forbindelse med, at vi tog app’en i brug, og her installerede vi også moduler fra Redia til samme formål (jeg kan desværre ikke huske præcis hvornår, men jeg vil skyde på at det var midt eller slut-juni eller start juli). Andre moduler har vi ikke installeret.

Der blev redigeret i flere af begivenhederne, mens andre blev slettet og oprettet på ny.​​​​​​​

#22 Updated by Rolf Madsen over 1 year ago

Place2book skriver at de har set fejlen i loggen hver gang de har kaldt Norddjurs.

#23 Updated by Rolf Madsen over 1 year ago

Er der nogen der har eksempler med link til fejl der lige er opstået i forbindelse med Place2book:

Vi kender allerede til, og har eksempler på:

#24 Updated by Rolf Madsen over 1 year ago

Tanja fra Hillerød skriver at det var:

"[...] alle vores arrangementer der så ud som den grøn-gule baggrund, og nu har vi jo rettet dem alle til, og lagt dem ind igen, så de skulle gerne se korrekte ud. [...]"

#25 Updated by Rolf Madsen over 1 year ago

Svar fra Jakob i Esbjerg på nedenstående tre spørgsmål:

-          Opdateret DDB CMS

-          Installeret nye 3. partsmoduler og hvilke var det i så fald

-          Redigéret i de begivenheder hvor indholdet er forsvundet fra?

 

Som jeg husker det er problemet opstået efter Jesper opdaterede CMS til seneste version. Jeg kan også se at han har prøvet at ”gen-opdatere” hjemmesiden, hvilket ikke gav nogen forbedring.

 

Jeg tror ikke vi har installeret nye 3. partsmoduler. Jesper er påpasselig med at gøre det, så det tvivler jeg på. Hvis det er sket er det i forbindelse med det Studieunivers han er ved at ”bygge”. Aarhus er projektledere på dette Studieunivers projekt, men jeg mener ikke at der er installeret noget, men kun indsat et tema og noogle tekster. Det skal jeg lige have helt afklaret når Jesper er tilbage fra ferie.

 

Problemet opstår når et arragement fremhæves til forsiden, så vises kort og forsvinder. Hvis vi undlader at fremhæve til forsiden kan vi få dem til at blive vist under arrangementerne. Det blev opdaget ved at vores kollega Pia undrede sig over at hun skulle have glemt at sætte et billede på et arrangement der skulle på forsiden, men glemme kan man jo. Så hun rettede det arrangement og opdaterede. ”Stille og roligt” forsvandt det så fra forsiden!

 

Dette betyder at vi pt. ikke viser arrangementer på forsiden.

#26 Updated by Martin Cording over 1 year ago

Vi har fundet ud af at der er nogle problemer med events som har tomt event tidspunkt:
https://github.com/ding2/ding2/pull/1157

Vi har endnu ikke fundet noget vedr. de slettede felter - det er som at finde en sort kat i et sort rum.

#27 Updated by Martin Cording over 1 year ago

  • Assignee changed from Martin Cording to Christian Vandel

Vi vil gerne have at I uploader dette modul til en af de sites som oplever problemet, så vi bedre kan se hvad der sker:

https://www.dropbox.com/s/xsgo5ek5iqhkfiq/d7_exception_handler.tar?dl=1

#28 Updated by Gitte Barlach over 1 year ago

Hej Martin 
Tak for opdateringen
Ifr. PR 1157: da du selv er inde på at det ikke løser sagen med de tomme felter, kan jeg så til at oprette en ny sag m. en kort beskrivelse + det tilhørende PR 1157?
 

#29 Updated by Christian Vandel over 1 year ago

@Martin: Jeg har lagt exception-handler-modulet på Hillerød, Vordingborg, Norddjurs og Hvidovre. Vi får se hvem der først får logget noget...

#30 Updated by Gitte Barlach over 1 year ago

  • Assignee changed from Christian Vandel to Martin Cording

#31 Updated by Christel Krabbenhøft over 1 year ago

  • Assignee changed from Martin Cording to Christian Vandel

Christian, er du rar at opdatere sagen, når der er nyt?

#32 Updated by Christian Vandel over 1 year ago

  • Assignee changed from Christian Vandel to Martin Cording

Hermed første eksempel på log efter tilføjelse af Martins exception-handler-modul (som jeg tidligere har hævdet er det ding_place2books opdatering af eventen, som trigger fejlen):

ddbwww-p93: Aug 20 23:25:13 http://hilbib.dk|1534800313|php|x.x.x.x|http://hilbib.dk/|https://hilbib.dk/|0||DrupalTraceBackException: PDOException#012SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'node-2872-3001-0-0-und' for key 'PRIMARY'#012#0 /var/aegir/platforms/hilleroed180702140054/includes/database/database.inc(2227): PDOStatement->execute(Array)#012#1 /var/aegir/platforms/hilleroed180702140054/includes/database/database.inc(697): DatabaseStatementBase->execute(Array, Array)#012#2 /var/aegir/platforms/hilleroed180702140054/includes/database/mysql/query.inc(36): DatabaseConnection->query('INSERT INTO {fi...', Array, Array)#012#3 /var/aegir/platforms/hilleroed180702140054/modules/field/modules/field_sql_storage/field_sql_storage.module(515): InsertQuery_mysql->execute()#012#4 /var/aegir/platforms/hilleroed180702140054/includes/module.inc(926): field_sql_storage_field_storage_write('node', Object(stdClass), 'update', Array)#012#5 /var/aegir/platforms/hilleroed180702140054/modules/field/field.attach.inc(1024): module_invoke('field_sql_stora...', 'field_storage_w...', 'node', Object(stdClass), 'update', Array)#012#6 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/ding_place2book/ding_place2book.module(340): field_attach_update('node', Object(stdClass))#012#7 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/ding_place2book/ding_place2book.module(203): ding_place2book_get_ticket_link(Object(stdClass))#012#8 /var/aegir/platforms/hilleroed180702140054/includes/module.inc(957): ding_place2book_node_view(Object(stdClass), 'teaser', 'da')#012#9 /var/aegir/platforms/hilleroed180702140054/modules/node/node.module(1441): module_invoke_all('node_view', Object(stdClass), 'teaser', 'da')#012#10 /var/aegir/platforms/hilleroed180702140054/modules/node/node.module(1336): node_build_content(Object(stdClass), 'teaser', 'da')#012#11 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/views/modules/node/views_plugin_row_node_view.inc(106): node_view(Object(stdClass), 'teaser')#012#12 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/views/plugins/views_plugin_style.inc(375): views_plugin_row_node_view->render(Object(stdClass))#012#13 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/views/plugins/views_plugin_style.inc(338): views_plugin_style->render_grouping_sets(Array)#012#14 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/views/theme/theme.inc(49): views_plugin_style->render(Array)#012#15 /var/aegir/platforms/hilleroed180702140054/includes/theme.inc(1125): template_preprocess_views_view(Array, 'views_view')#012#16 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/views/plugins/views_plugin_display.inc(2646): theme('views_view', Array)#012#17 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/views/includes/view.inc(1264): views_plugin_display->render()#012#18 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/views/plugins/views_plugin_display.inc(2718): view->render()#012#19 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/views/includes/view.inc(1368): views_plugin_display->preview()#012#20 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/ctools/views_content/plugins/content_types/views_panes.inc(287): view->preview()#012#21 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/ctools/includes/content.inc(281): views_content_views_panes_content_type_render('ding_event-ding...', Array, Array, Array, '')#012#22 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/panels/plugins/display_renderers/panels_renderer_standard.class.php(558): ctools_content_render('views_panes', 'ding_event-ding...', Array, Array, Array, Array)#012#23 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/panels/plugins/display_renderers/panels_renderer_standard.class.php(494): panels_renderer_standard->render_pane_content(Object(stdClass))#012#24 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/panels/plugins/display_renderers/panels_renderer_standard.class.php(473): panels_renderer_standard->render_pane(Object(stdClass))#012#25 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/panels/plugins/display_renderers/panels_renderer_standard.class.php(389): panels_renderer_standard->render_panes()#012#26 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/panels/plugins/display_renderers/panels_renderer_standard.class.php(360): panels_renderer_standard->render_layout()#012#27 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/panels/panels.module(747): panels_renderer_standard->render()#012#28 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/panels/panels.module(1112): panels_display->render(Object(panels_renderer_standard))#012#29 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/panels/plugins/task_handlers/panel_context.inc(328): panels_render_display(Object(panels_display), Object(panels_renderer_standard))#012#30 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/ctools/includes/context-task-handler.inc(121): panels_panel_context_render(Object(stdClass), Array, Array)#012#31 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/ctools/includes/context-task-handler.inc(44): ctools_context_handler_render_handler(Array, Array, Object(stdClass), Array, Array)#012#32 /var/aegir/platforms/hilleroed180702140054/profiles/ding2/modules/contrib/ctools/page_manager/plugins/tasks/page.inc(322): ctools_context_handler_render(Array, Array, Array, Array)#012#33 /var/aegir/platforms/hilleroed180702140054/includes/menu.inc(531): page_manager_page_execute('ding_frontpage')#012#34 /var/aegir/platforms/hilleroed180702140054/index.php(21): menu_execute_active_handler()#012#35 {main}#012INSERT INTO {field_revision_field_ding_event_location} (entity_type, entity_id, revision_id, bundle, delta, language, field_ding_event_location_country, field_ding_event_location_administrative_area, field_ding_event_location_sub_administrative_area, field_ding_event_location_locality, field_ding_event_location_dependent_locality, field_ding_event_location_postal_code, field_ding_event_location_thoroughfare, field_ding_event_location_premise, field_ding_event_location_sub_premise, field_ding_event_location_organisation_name, field_ding_event_location_name_line, field_ding_event_location_first_name, field_ding_event_location_last_name, field_ding_event_location_data) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2, :db_insert_placeholder_3, :db_insert_placeholder_4, :db_insert_placeholder_5, :db_insert_placeholder_6, :db_insert_placeholder_7, :db_insert_placeholder_8, :db_insert_placeholder_9, :db_insert_placeholder_10, :db_insert_placeholder_11, :db_insert_placeholder_12, :db_insert_placeholder_13, :db_insert_placeholder_14, :db_insert_placeholder_15, :db_insert_placeholder_16, :db_insert_placeholder_17, :db_insert_placeholder_18, :db_insert_placeholder_19)#012Query args: [node][2872][3001][ding_event][0][und][DK][][][][][][][][][][][][][] in _d7_exception_handler_catch() (line 40 of /var/aegir/platforms/hilleroed180702140054/sites/all/modules/d7_exception_handler/d7_exception_handler.module).


 

#33 Updated by Martin Cording over 1 year ago

  • Status changed from Needs analysis to Needs code review
  • Assignee changed from Martin Cording to Gitte Barlach

Fejlen opstår når der er samtidige field save rutiner, initieret af et Place2book request. Det er ikke muligt at have simultane operationer af denne type.
Dette kan opstå der er nok trafik, hvor en bruger initierer en P2b request som forsøger at opdatere et felt på en node, og på samme tid (før feltet er opdateret fra af den første bruger), en anden bruger initierer samme logik - for samme felt på noden.

I og med database operations ikke er asynkrone bliver vi fanget i et pitfall.

Eftersom hver record i en specifik tabel er defineret via primary keys, kan der ikke eksistere duplikater af værdier som matcher hinanden 100%.
Hvilket er præcis hvad der sker, hvis mere end 1 bruger prøver at se et event på samme tid - der er en chance for at det sker på nogenlunde samme tid.

Jeg tror dette commit er et hurtigt workaround, eftersom jeg ikke mener at der er behov for at requeste P2b hver gang der vises et event.
Ændringen prøver at låse en record i et sekund, for at lade en opdatering færdigudføres: https://github.com/ding2/ding2/pull/1157.

#34 Updated by Kasper Garnæs over 1 year ago

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

Jeg er enig i Martins teori vedr. hvad problemet skyldes. I så fald er er det opstået i forbindelse med løsningen af #3538.

Jeg har spørgsmål vedr. hvordan dette er løst teknisk.

#35 Updated by Martin Cording over 1 year ago

  • Assignee changed from Martin Cording to Kasper Garnæs

#3538 blev oprettet fordi der var problemer med feltet field_ding_event_ticket_link og dem der bruge Place2book.

I første omgang blev der lavet en løsning, således at når en node blev oprettet eller opdateret - så blev P2b-linket migreret til field_ding_event_ticket_link (som vises på noden og i app'en).

Efterfølgende var problemet at der var rigtig mange noder som allerede eksisterede og som ikke fik flyttet dette felt.
Det blev løst ved at tjekke for dette under visningen af noden (ding_place2book_node_view), og ikke kun opret (ding_place2book_node_insert) og opdater (ding_place2book_node_update).

Det er uhensigtsmæssigt at et node view ændrer database records. Jeg foreslår at vi flytter det til en hook_update() og kører alle noder igennem ved opdatering istedet.

#36 Updated by Gitte Barlach over 1 year ago

  • Status changed from Reviewed - Needs info/rework to Needs code review

tak for forklaring Martin; besvarer det de spørgsmål du har @Kasper ?

#37 Updated by Kasper Garnæs over 1 year ago

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

Jeg foreslår at vi flytter det til en hook_update() og kører alle noder igennem ved opdatering istedet.

Enig.

#38 Updated by Rolf Madsen over 1 year ago

  • Related to Bug #3751: Fejl i arrangementer med fri entre og med billetsalg (også staging) added

#39 Updated by Gitte Barlach over 1 year ago

Relateret til #3755

#40 Updated by Martin Cording over 1 year ago

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

PR er opdateret med løsningen.

#41 Updated by Kasper Garnæs over 1 year ago

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

Jeg har spørgsmål til hvordan løsningen er blevet implementeret.

#42 Updated by Rolf Madsen over 1 year ago

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

#43 Updated by Rolf Madsen over 1 year ago

  • Status changed from Reviewed - Needs info/rework to Closed

Tak for det Martin!

Dette issue lukkes hermed.

#44 Updated by Martin Cording over 1 year ago

  • Status changed from Closed to Reviewed - Needs info/rework

Rolf jeg tror det er en fejl? Der ligger åbent PR her som SKAL ind ;-)

#45 Updated by Gitte Barlach over 1 year ago

Tak for det, Martin, ja det var en fejl; sagen skal ikke lukkes. 
Kan du sige noget om hvornår I kan vende tilbage på denne?

#46 Updated by Rolf Madsen over 1 year ago

Det var vist et andet issue der skulle have været lukket ... Beklager ulejligheden!

#47 Updated by Martin Cording over 1 year ago

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

PR opdateret - der er ingen grund til at opdatere gamle noder mere.

#48 Updated by Christel Krabbenhøft over 1 year ago

  • Target version changed from Release 29-2 - Bugfixes (Inlead) to Release 29-1 - Lister (7.x-4.4.0)

#49 Updated by Kasper Garnæs over 1 year ago

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

Ændringen er OK, hvis forretningen kan acceptere det workaround der bliver nævnt andetsteds hvor man manuelt skal ind og opdatere indhold med manglende P2B info.

Derudover har jeg et forslag til forbedring: https://github.com/easySuite/ding2/pull/477.

#50 Updated by Christel Krabbenhøft over 1 year ago

  • Status changed from Reviewed to Need more info
  • Assignee changed from Gitte Barlach to Kasper Garnæs

Hej Kasper - vil du ikke uddybe din sidste kommentar? Vi har svært ved at vurdere, hvad workarounden konkret vil betyde for bibliotekerne?

#51 Updated by Martin Cording over 1 year ago

Kasper, jeg skal også lige forstå. Der bør ikke være nogen events som mangler P2b info.
De events som tidligere lå i batchen, ville få P2B id = -1 - og ville blive håndteret som et event der ikke vedligeholdes hos P2b.

Alt dette handler om events som er blevet oprettet før Place2book modulet er blevet aktiveret.

Jeg har kommenteret på dit PR, som vil give problemer igen!

#52 Updated by Kasper Garnæs over 1 year ago

  • Status changed from Need more info to Needs code review
  • Assignee changed from Kasper Garnæs to Martin Cording

Martin: OK. Jeg havde forstået det som om at der var behov for at gennemføre en opdatering af de events som lider af dette problem, da det i første omgang var omdrejningspunktet for reviewet. Da denne opdatering efterfølgende var blevet fjernet uden yderligere forklaring, antog jeg at I i stedet for havde valgt en løsning, således at fejlen ikke længere optrådte, men at koblingen til place2book nu skulle aktiveres redaktionelt. Det kan jeg så forstå nu ikke er tilfældet. Beklager forvirringen.

Jeg har opdateret mit eget PR.

#53 Updated by Martin Cording over 1 year ago

  • Assignee changed from Martin Cording to Kasper Garnæs

Approved and merged.

#54 Updated by Kasper Garnæs over 1 year ago

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

Godkendt og merged.

#55 Updated by Simon Holt over 1 year ago

@Martin har lige et enkelt spørgsmål. Det er en kompliceret sag at læse igennem synes jeg, så beklager hvis jeg har misforstået noget.

jeg kan ikke helt gennemskue hvordan arrangementer oprettet i place2book, før ticket_link feltet blev introduceret, får migreret p2b link over i i det nye felt (eller om det overhovedet er nødvendigt)? Jeg er med på alt det med at det ikke er nødvendigt med den første batch i lavede, men er det ikke stadig nødvendigt at flytte linket over? 

Min kollega har f.eks. lige oprettet en masse arrangementer til det kommende efterårsprogram i en version uden ticket_link feltet. Når jeg merger upstream, skal vi så ind og trykke opdater på hver eneste node, for at få flyttet p2b link ned i ticket_link feltet?

Jeg kan jo se i PR, at den prøver at opdatere ticket_link feltet ved hook_update/insert, så det får mig selvfølgelig til at tro, at det er nødvendigt at flytte linket over. Men da det ikke længere sker i hook_node_view() (hvilket selvfølgelig er rigtig godt hehe), og der ikke er noget opdateringsfunktion der sørger for det, kan jeg ikke rigtig se anden udvej, end at vi skal ind og trykke opdater på eksisterende arrangementer?

 

#56 Updated by Martin Cording over 1 year ago

Jo, det er nødvendigt at flyttet linket over :-)

ding_place2book_get_ticket_link() bliver kaldt på linie 245 af ding_place2book_node_insert() (nye noder) og linie 302 af ding_place2book_node_update() (eksisterende noder).

ding_place2book_get_ticket_link() migrerer linket til field_ding_event_ticket_link på linie 358.

#57 Updated by Simon Holt over 1 year ago

Ah ok, tak for svar :)

Men har det som om jeg overser et eller andet her, hehe. Prøver lige igen:

ding_place2book_node_update() bliver vel kun kaldt, når man opdaterer en node? Vi har i hvert fald 100 arrangementer hvor det skal gøres. Ville det ikke være smartere med en opdateringsfunktion?

#58 Updated by Martin Cording over 1 year ago

Oh - jo altså hvis der er "gamle" noder som skal have flyttet link automatisk, så bør der laves en hook_update() der kører ding_place2book_get_ticket_link() på alle noder med brug af ding_place2book_get_id()!

#59 Updated by Simon Holt over 1 year ago

Vil mene vi bør lave dette for at undgå at skulle opdatere en potentielt stor mængde arrangmenter manuelt.

#60 Updated by Rolf Madsen over 1 year ago

@Simon, Ville det være tilstrækkeligt at gå ind under /content, filtrere på content type event også execute publish content?

Jeg mener den rigtige løsning er at undgå at bibliotekerne skal opdatere deres arrangementer for at få "Bestil billet(ter)" knappen, men af hensyn til at få releasen udrullet, er vi nok nødt til at afsøge alle muligheder. 

#61 Updated by Simon Holt over 1 year ago

@Rolf Ja, det vil virke. Med "node_publish_action" vil alle arrangementerne til sidst komme igennem node_save() hvilket udløser ding_placebooks' hook_node_update() implementation.

Der burde heller ikke være noget problem med timeout, da bulk-operations i indholdsoversigten vil beskytte imod dette.

#62 Updated by Martin Cording over 1 year ago

Det er jeg ikke sikker på Simon - jf. inline dokumentationen så vil de kun blive opdateret hvis der er en form alter:

function ding_place2book_node_update($node) {
  if ($node->type == "ding_event") {
    // Continue only if node contains place2book data - they will not be there
    // when performing a bulk update, updating via scheduler module etc.
    if (empty($node->place2book)) {
      return;
    }

#63 Updated by Simon Holt over 1 year ago

Og det står lige der i kommentaren, at den netop ikke gør det ved bulk updates. Den havde jeg overset.

En opdateringsfunktion er også at foretrække ;)

#64 Updated by Tanja Kate Jensen over 1 year ago

På Hillerød Bibliotekerne har jeg efter aftale med Christel Elbrønd Krabbenhøft netop testet template "ding2-7.x-4.4.1-rc1-kun-test", og kan ikke finde noget usædvanligt ved begivenhederne (andet end at der stadig vises -nothing- under hvert gratis arrangement - se vedhæftede -  men jeg ved ikke om det hører under dette issue). Jeg testede ved blot at klikke ind på alle vores arrangementer og tjekke dem igennem - er der andet, I vil have jeg gør, for at teste?

#65 Updated by Christel Krabbenhøft over 1 year ago

Tusind tak, Tanja! Det er rigtig fint for nu - vi vender muligvis tilbage og beder om mere testhjælp:-)

#66 Updated by Martin Cording over 1 year ago

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

Hold da op hvor er dette modul komplekst efterhånden.. Jeg glæder mig til at #1404 bliver en realitet :)

Her er PR som gemmer alle events med P2b status: https://github.com/ding2/ding2/pull/1171

#67 Updated by Rolf Madsen over 1 year ago

  • Status changed from Reviewed - Needs info/rework to Needs code review

#68 Updated by Kasper Garnæs over 1 year ago

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

Reviewed. Jeg har et par mindre kommentarer.

#69 Updated by Martin Cording over 1 year ago

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

Fixed.

#70 Updated by Kasper Garnæs over 1 year ago

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

Godkendt og merged.

#71 Updated by Martin Cording over 1 year ago

Tester @Tanja igen? :)

#72 Updated by Christel Krabbenhøft over 1 year ago

Ja, Tanja tester i morgen formiddag. Jesper fra Esbjerg har allerede testet, og meldt følgende tilbage:

———

Nu har jeg lige lavet en hurtig test.

Det ser ud til at det der var kommet til at virke før, stadig virker.

Desuden er ”Bestil billetter”-knappen også kommet med og det ser også ud til at fungere efter hensigten.

#73 Updated by Tanja Kate Jensen over 1 year ago

Det virker fint, når jeg tester, og billetknappen ser fin ud / -nothing- tag er væk. 

#74 Updated by Martin Cording over 1 year ago

Det lyder godt @Tanja; er der nogen mulighed for at du kan teste følgende og vedhæfte et screenshot af selve eventen i hvert tilfælde:

  • Se hvordan det ser ud på et arrangement som blev oprettet før I aktiverede modulet (gå til i Content og filtrer efter Begivenhed og sorter efter dato - vælg den ældste du kan finde)
  • Tjekke hvordan det ser ud med forskellige status'er, herunder:
    • event-over, når arrangmentet er overstået
    • order-waiting-list, når der er venteliste
    • no-tickets-left, når der er udsolgt
    • closed-admission, når der er lukket for salg
    • sale-not-started, når salget endnu ikke er startet

#75 Updated by Tanja Kate Jensen over 1 year ago

o    event-over, når arrangmentet er overstået - se vedhæftede 
o    order-waiting-list, når der er venteliste - den bruger vi ikke, vi laver papirventeliste
o    no-tickets-left, når der er udsolgt - se vedhæftede
o    closed-admission, når der er lukket for salg - Jeg er ikke sikker på at vi bruger denne - hvordan adskiller den sig fra no_tickets_left?
o    sale-not-started, når salget endnu ikke er startet - Vores billetter går i salg fast i begyndelsen af hhv. januar og august, så pt. har vi ingen arrangementer på webben, der ikke er gået i salg endnu :/
 

#76 Updated by Gitte Barlach over 1 year ago

Har testet på upgrade-fbs:

Jeg har netop opdateret nogle arrangementer og givet dem en ny dato for afholdelse. De står med "Billetsalg lukket":
https://upgrade-fbs.ddbcms.dk/arrangementer/boern/jazz-med-bejamin-koppel-og-casper-rongsted
https://upgrade-fbs.ddbcms.dk/arrangementer/boern/moed-forfatteren-sara-blaedel
https://upgrade-fbs.ddbcms.dk/arrangementer/internet-undervisning/klassisk-musik-i-godt-selskab

Jeg lægger mærke til der står "Tickets" øverst opppe til venstre, når jeg har åbnet pågældende arrangementer til redigering. Se vedlagte skærmdump.

OBS Dette  gælder ikke ny-oprettede arrangementer.
 

#77 Updated by Gitte Barlach over 1 year ago

@Tanja: har du oplevet fejl som den jeg beskriver ovenfor på jeres test-installation? 

#78 Updated by Martin Cording over 1 year ago

  • Status changed from Technical test to Needs code review

Ok, skal vi lige have det med også? https://github.com/ding2/ding2/pull/1185

Alle status bliver nu vist på samme måde:

#79 Updated by Rolf Madsen over 1 year ago

Hvis vi har forstået https://github.com/ding2/ding2/pull/1176 korrekt så er den af kosmetisk karakter, selv om det virkelig bliver meget pænere.

Af hensyn til tidsplanen flyttes den derfor til Release 30.

https://platform.dandigbib.org/issues/3819 er oprettet til formålet.

#80 Updated by Martin Cording over 1 year ago

  • Status changed from Needs code review to Technical test

#81 Updated by Gitte Barlach over 1 year ago

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

Okay - når vi ser bort fra seneste PR der behandles i #3819, så er sagen her testet på upgrade-fbs med 4.4.1-rc2, samt på hhv. Esbjerg og Hillerød stg. 
Rettelserne ser ud til at virke efter hensigten. Sætter derfor sagen til 'resolved (tag version)'

#82 Updated by Christel Krabbenhøft over 1 year ago

  • Status changed from Resolved (tag version) to Ready for development
  • Assignee changed from Gitte Barlach to Martin Cording

Hej Martin. Kan I kigge på denne ASAP? Selvom vi havde testet og godkendt på Esbjerg og Hillerød, så optræder fejlen i seneste release (29.1). Allerød, Stevns og Fredensborg har foreløbig indberettet den.

#83 Updated by Martin Cording over 1 year ago

  • Status changed from Ready for development to Need more info
  • Assignee changed from Martin Cording to Christel Krabbenhøft

Ja, det kan vi godt. Hvad er det som ikke virker/fejler eller som ikke er løst?

#84 Updated by Christel Krabbenhøft over 1 year ago

  • Assignee changed from Christel Krabbenhøft to Martin Cording

#85 Updated by Martin Cording over 1 year ago

  • Assignee changed from Martin Cording to Christel Krabbenhøft

Og er vi sikre på at deres site er opdateret? Deres staging site ser fin ud: https://alleroed-stg.ddbcms.dk/arrangementer/litteratur/knud-romer-kort-over-paradis.

#86 Updated by Christel Krabbenhøft over 1 year ago

  • Assignee changed from Christel Krabbenhøft to Martin Cording

Jeg deler din undren, Martin, men ja de er opdateret - jeg ringede med det samme til Allerød og bad dem bekræfte. De blev opdateret i formiddags.

#87 Updated by Martin Cording over 1 year ago

  • Assignee changed from Martin Cording to Christel Krabbenhøft

Okay, hvilken version kører deres staging site så?

Vi skal bruge et dump af deres produktionssite for at undersøge nærmere.

#88 Updated by Tue Gaston over 1 year ago

  • Related to Bug #3538: Billetkøbeknapper til Place2Book og titelbilleder mangler på alle arrangementer added

#89 Updated by Tue Gaston over 1 year ago

  • Related to Bug #3741: Manglende billetknapper hos Gentofte og Gladsaxe Biblioteker added

#90 Updated by Tue Gaston over 1 year ago

  • Is duplicate of deleted (Bug #3608: Arrangementer som ikke kræver billetsalg får "nothing" i billetknapsfeltet)

#91 Updated by Tue Gaston over 1 year ago

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

#92 Updated by Christel Krabbenhøft over 1 year ago

  • Assignee changed from Christel Krabbenhøft to Martin Cording

Hej Martin. Det har du på mail nu!

#93 Updated by Christian Vandel over 1 year ago

Allerøds prod-site kører 7.x-4.4.1

-dev-sitet kører 7.x-4.4.0-rc1

#94 Updated by Martin Cording over 1 year ago

  • Status changed from Need more info to Needs code review
  • Assignee changed from Martin Cording to Gitte Barlach

Fixed: https://github.com/ding2/ding2/pull/1176

Der manglede lige en "#" i en kodelinje..
(I samme PR løses https://platform.dandigbib.org/issues/3704#note-76 også)

#95 Updated by Jesper Kristensen over 1 year ago

Vi har som sagt oplevet det samme problem på aakb.dk (altså med at knapper forsvinder), men er nu skiftet til pretix.

Det viste sig at vores knap blev indsat på hook_entity_load og virkede fint ind til page cache/varnish expire og når event siden så skulle bygges igen blev hook_entity_load ikke kaldt, da selv node'en var i entity_cache.

Så vores løsning var at bruge begge hooks: hook_entity_load() + hook_entitycache_node_load()

Kan ikke huske om p2b bruger hook_entity_load(), men måske er det der det går galt.

#96 Updated by Jesper Kristensen over 1 year ago

Reviewed og godkendt.

Men rettet disse HTML ændringer ved at bestil knap'erne ikke altid er der?

#97 Updated by Jesper Kristensen over 1 year ago

  • Status changed from Needs code review to Reviewed

#98 Updated by Rolf Madsen over 1 year ago

  • Status changed from Reviewed to Needs code review

Det betyder vi kommer til at mangle oversættelser ...

 

@Martin vil nedenstående virke hvis jeg tilføjer det til masteroversættelsen?

Pga knappernes karakter af – netop, ja – en knap, ønsker vi så korte tekster som overhovedet muligt:


msgid "Book a ticket"
msgstr "Bestil billet(ter)"

msgid "Waiting list open"
msgstr "Ventelisten er åben"

msgid "Ticket sale has not yet started for this event"
msgstr "Billetbestilling åbner snart”

msgid "Sold out"
msgstr "Der er ikke flere billetter”

msgid "Not open for ticket sale"
msgstr "Billetbestilling ikke mulig” [OBS! Denne afventer lige svar fra @Martin - hvad udløser egentlig denne besked? Hvad er forskellen på denne og "Ticket sale has not yet started for this event"?]

msgid "The event has already taken place"
msgstr "Arrangementet er slut"

#99 Updated by Rolf Madsen over 1 year ago

Hvad er forskellen på:

  1. "Ticket sale has not yet started for this event"
  2. "Not open for ticket sale"

Hvilken status har gratis arrangementer?

#100 Updated by Christel Krabbenhøft over 1 year ago

  • Status changed from Needs code review to Need more info

Martin, lige for at være HELT sikker: Betyder den ene rettelse i stylingen, at knapperne lader være med at forsvinde? Og vil du ikke også svare Rolfs kommentar #98.

#101 Updated by Christel Krabbenhøft over 1 year ago

  • Assignee changed from Gitte Barlach to Martin Cording

#102 Updated by Martin Cording over 1 year ago

Jeg ved ikke lige hvad der skete med alle kommentarer i dette issue?
Der er ikke lavet om i nogle oversættelser eller lignende.

Denne PR gør følgende:

  • Sørger for at et kunne kun er de første 20 noder der bliver processed
  • Omringer "Tickets" i en <div> så den ikke flagrer oppe i venstre hjørne

Issue #3819 håndterer de andre nævnte look n' feel ændringer. 

#103 Updated by Martin Cording over 1 year ago

  • Assignee changed from Martin Cording to Rolf Madsen

#104 Updated by Gitte Barlach over 1 year ago

Hej Martin
er det rettelser der vil fungere hvis de patches på sites, der allerede er opgraderede og dermed mangler billetknapper? vil knapperne dermed komme igen?
 

#105 Updated by Martin Cording over 1 year ago

  • Assignee changed from Rolf Madsen to Gitte Barlach

Dette vil kun have effekt på dem som ikke allerede har kørt opdateringen!
Dem som har kørt opdateringen, og som opdaterer igen - vil ikke få løst problemet.

Er der lavet en release som er rullet på produktion med denne fejl?
I så fald; skal det løses for dem som allerede har opgraderet?

#106 Updated by Gitte Barlach over 1 year ago

vedr.:"Er der lavet en release som er rullet på produktion med denne fejl?"
ja, desværre
Vedr.: "I så fald; skal det løses for dem som allerede har opgraderet?"
Ja, tak, meget gerne!

#107 Updated by Gitte Barlach over 1 year ago

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

#108 Updated by Marianne Sivertsen over 1 year ago

Jeg ved ikke, om I vil have dette som en separat sag - eller som kommentar her:

Ud over at billetknapperne var forsvundet efter vi blev opgraderet forleden (jeg har i mellemtiden fixet dem manuelt) - så medfører et passivt arrangement følgende billetbesked:

'Billetsalg lukket'.

Og jeg skal have det ud i Kultunaut, så derfor er jeg nødt til at slå P2B til.

#109 Updated by Martin Cording over 1 year ago

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

#110 Updated by Gitte Barlach over 1 year ago

  • Assignee changed from Gitte Barlach to Jesper Kristensen

#111 Updated by Gitte Barlach over 1 year ago

Tak for PR, Martin !
Har du kigget på Jespers kommentar https://platform.dandigbib.org/issues/3704#note-95 ?
 

#112 Updated by Martin Cording over 1 year ago

Ja, men jeg er ikke klar over hvordan det er relevant i dette tilfælde?

#113 Updated by Gitte Barlach over 1 year ago

Hej Martin 
Den fejl vi have på aakb.dk gav sig udslag i at billetknapperne forsvandt på sitet. De var der godt nok når man oprettede et event, men senere forsvandt de bare igen. Jesper løste det  som sagt ved at bruge begge hooks: hook_entity_load() + hook_entitycache_node_load()

Da noget af problemet i denne sag ligeledes giver sig udslag i at noget indhold forsvinder, var det blot et indspark til hvad den reelle årsag kunne være. 

Hvis I har analyseret på det, og kan sige at det (entity cache) helt klart ikke har noget med sagen at gøre, så lad os bare komme videre. Jeg ville bare være sikker på at I havde set forslaget. 
 

#114 Updated by Martin Cording over 1 year ago

Denne sag handler om at feltindhold mistes og hvordan det håndteres - der er ikke nogen implemetering af entity cache i nogen af de PRs som er nævnt i denne sag.

#115 Updated by Simon Holt over 1 year ago

p2b anvender ikke hook_entity_load() men hook_node_view(). Måske er der derfor ikke problemer med entitycache?

#116 Updated by Gitte Barlach over 1 year ago

tak @Simon, det var lige den forklaring jeg ledte efter. 

 

#117 Updated by Jesper Kristensen over 1 year ago

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

Reviewed og godkendt

#118 Updated by Jesper Kristensen over 1 year ago

  • Status changed from Reviewed to Technical test

Merged

#119 Updated by Carsten Vilhelmsen over 1 year ago

Lige et indspark fra Tårnby. Vi oplever vi at udsolgte arrangementer får teksten "nothing" i stedet for udsolgt

https://taarnbybib.dk/arrangementer/litteratur/forfattersamtale-med-morten-pape 

#120 Updated by Martin Cording over 1 year ago

Carsten er arrangementet markeret som udsolgt i P2b, eller er det markeret som åben med 0 billetter?

#121 Updated by Carsten Vilhelmsen over 1 year ago

Det er markeret som udsolgt

#122 Updated by Stefan Søndervang over 1 year ago

Der er blevet skrevet om -nothing- fejlen i nyheden om den nyeste release

"Billetstatus ved udsolgte og passive arrangementer er lige nu misvisende. Ved passive arrangementer er billetstatus “Billetsalg lukket”, og ved udsolgte billetter står der “-nothing-”. Denne fejl ved billetstatus fjerner ingen vigtig funktionalitet. Da statusbeskrivelserne ikke er svært misvisende, bliver fejlen rettet i næste opgradering (Release 29.2 - bugfixes), som vi forventer at have klar til release ultimo oktober.

Vi anbefaler ved passive arrangementer, at I skriver i brødteksten, at arrangementet er åbent uden billet."

#123 Updated by Rolf Madsen over 1 year ago

  • Has duplicate Bug #3900: Manglende billetknap på udsolgte arrangementer added

#124 Updated by Rolf Madsen over 1 year ago

  • Has duplicate deleted (Bug #3900: Manglende billetknap på udsolgte arrangementer)

#125 Updated by Rolf Madsen about 1 year ago

  • Target version changed from Release 29-1 - Lister (7.x-4.4.0) to Release 30-1 - Place2Book (7.x-4.7.0)

Jeg flytter dette issue til Release 30-1 med henblik på test.

#126 Updated by Stefan Søndervang about 1 year ago

  • Kategorier Integration - Place2book (Arangementer) added

#127 Updated by Gitte Barlach 11 months ago

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

testet jvf. #1404. Herunder at billet-knapper har korrekt status, samt at feltindhold ikke forsvinder. 

Also available in: Atom PDF