Project

General

Profile

Bug #2927

Rediger-knapper m.v. er ikke synlige for webmaster i “4.0.0-kun-test”

Added by Anonymous over 2 years ago. Updated about 2 years ago.

Status:
Resolved (tag version)
Priority:
High
Assignee:
Estimated time:
URL med eksempel:
Kategorier:
Inspiration - Nyheder, Inspiration - Arrangementer, Biblioteksinfo - Sider, Inspiration - Sektioner og temaer

Description

Det ser ud til, at i  “4.0.0-kun-test” hvis man har rettigheden webmaster, så forsvinder knapperne til redigering m.v. op under den vandrette menu på nyheder, arrangementer, grupper og sider.

Det ser ud til, at hvis man er localadmin eller editor, og der ikke er en undermenu, så er knapperne synligt - men hvis der er en undermenu, forsvinder knapperne op under den.

Det gør ikke nogen forskel, om man klapper webmaster menulinjen sammen (den sorte) - men det giver andre problemer, som jeg beskriver i et andet issue.

nyhed - webmaster.PNG (469 KB) nyhed - webmaster.PNG Anonymous, 10/10/2017 12:09 PM
nyhed - editor.PNG (447 KB) nyhed - editor.PNG Anonymous, 10/10/2017 12:09 PM
arrangement - redaktor.PNG (577 KB) arrangement - redaktor.PNG Anonymous, 10/10/2017 12:18 PM
arrangement - webmaster.PNG (898 KB) arrangement - webmaster.PNG Anonymous, 10/10/2017 12:18 PM
gruppe - editor.PNG (818 KB) gruppe - editor.PNG Anonymous, 10/10/2017 12:18 PM
nyhed - editor.PNG (447 KB) nyhed - editor.PNG Anonymous, 10/10/2017 12:18 PM
nyhed - webmaster.PNG (469 KB) nyhed - webmaster.PNG Anonymous, 10/10/2017 12:18 PM
side med undermenu - editor.PNG (687 KB) side med undermenu - editor.PNG Anonymous, 10/10/2017 12:18 PM
gruppe - webmaster.PNG (797 KB) gruppe - webmaster.PNG Anonymous, 10/10/2017 12:18 PM
side - webmaster.PNG (684 KB) side - webmaster.PNG Anonymous, 10/10/2017 12:18 PM
webmaster profil mangler knapper.PNG (124 KB) webmaster profil mangler knapper.PNG Anonymous, 10/10/2017 03:46 PM
nyhed - redaktør - ladmin login.JPG (156 KB) nyhed - redaktør - ladmin login.JPG Sesse Nisted Atkin, 10/11/2017 01:37 PM
secure_permissions.PNG (41.4 KB) secure_permissions.PNG Anonymous, 10/12/2017 09:22 AM
admin-menu.PNG (40.3 KB) admin-menu.PNG Anonymous, 10/12/2017 09:52 AM
admin login dobbelmenu.JPG (78.6 KB) admin login dobbelmenu.JPG Tanja Kate Jensen, 10/13/2017 11:46 AM
local admin login dobbeltmenu.JPG (78.6 KB) local admin login dobbeltmenu.JPG Tanja Kate Jensen, 10/13/2017 11:46 AM
Dobbeltmenu.jpg (313 KB) Dobbeltmenu.jpg Katrine Isaksen, 10/31/2017 01:08 PM
dump.jpg (164 KB) dump.jpg Michael W. Christoffersen, 10/31/2017 02:38 PM
Screenshot login som admin i nyt design.png (635 KB) Screenshot login som admin i nyt design.png Dobbelt menubar ved login Katrine Isaksen, 11/06/2017 01:11 PM

Related issues

Related to DDB CMS - Bug #2929: webmasterpanel forsvinder i “4.0.0-kun-test”, hvis man kommer til at trykke på grå pilResolved
Related to DDB CMS - Bug #3020: Rediger knap skjules hvis hoved- og sekundær menu er for lang og sekundær menu skjules hvis admin menu vises og genvejsmenu foldes sammenClosed
Related to DDB CMS - Bug #2184: Toolbar (menu) virker ikke for local editor og guest bloggerClosed

History

#1 Updated by Rolf Madsen over 2 years ago

  • Status changed from New to Ready for development
  • Assignee set to Kristian Pedersen
  • Target version set to DDB CMS 2017 1.1 opgradering (B14 sprintbacklog)

#2 Updated by Anonymous over 2 years ago

Min "webmaster" bruger har rettighederne "administrator" og local administrator"

Det samme problem med manglende knapper gælder i øvrigt brugerprofilen for webmaster.

#3 Updated by Simon Holt over 2 years ago

Hej Tina,

Har den ikke andre roller end "administrator" og local administrator"?

Ser ud som om du er ramt af problematikken med dobbelt admin bar. Det kan ske hvis man kombinerer rollerne, men burde egentlig ikke ske de roller du nævner.

#4 Updated by Anonymous over 2 years ago

Nej ikke andre roller, altså ud over authenticated user.

Jeg har talt med Sesse fra Ishøj, og hun har også problemet, men de er redaktør-bibliotek og hun oplever det vist med rettighederne "local administrator"

#5 Updated by Sesse Nisted Atkin over 2 years ago

Vores ladmin har rettighederne editor og local administrator. Og der har vi problemet. Vi har ikke problemet på en som har editor. Vi er som Tina skriver et redaktør bibliotek.

 

#6 Updated by Kristian Pedersen over 2 years ago

  • Status changed from Ready for development to Need more info
  • Assignee changed from Kristian Pedersen to Gitte Barlach

Er det muligt at få et af de log-ins der oplever problemet og et link til det pågældende site?

#7 Updated by Simon Holt over 2 years ago

Vores ladmin har rettighederne editor og local administrator

Ja, dette vil resultere i to admin bars, der overlapper hinanden og skaber problemer. Dette skyldes, at editor har tilladelsen 'access toolbar' mens local administrator har tilladelsen 'access administration menu'.

Den løsning vi har lavet, er slet ikke gearet til at I blander rollerne. Så vi skal enten finde ud at sætte tiladelserne op, så det ikke er nødvendigt at kombinere rollerne eller også skal vi have fundet en anden løsning til problematikken beskrevet i disse sager: #2596 og #2184.

#8 Updated by Simon Holt over 2 years ago

Nej ikke andre roller, altså ud over authenticated user.

Det lyder mystisk. Har du adgang til /admin/config/people/secure_permissions?

#9 Updated by Rolf Madsen over 2 years ago

Det undrer også mig at jeg med admin rollen i https://upgrade-fbs.ddbcms.dk/ får vist:

Den dobbelte menu vises altså ikke længere for admin brugeren, hvilket vi ellers havde accepteret at leve med da den kun blev vist for netop admin rollen. Det er ikke noget de langt mere benyttede roller skal leve med.

Derudover kan jeg ikke længere folde undermenuerne ud ved mouse-over ...

#10 Updated by Sesse Nisted Atkin over 2 years ago

Den løsning vi har lavet, er slet ikke gearet til at I blander rollerne. Så vi skal enten finde ud at sætte tiladelserne op, så det ikke er nødvendigt at kombinere rollerne.

Hvad kan en editor som en local administrator ikke kan? for det giver mest mening for mig at at jeg kan alt på et login, så jeg ikke skal logge ind og ud og huske flere login. 

#11 Updated by Rolf Madsen over 2 years ago

Lokal administratorrollen skulle gerne have flere og ikke færre tilladelser end editor rollen.

#12 Updated by Anonymous over 2 years ago

Jeg læste i en tråd på FB, at der var en eller anden bug der gjorde, at admin og local admin burde kombineres for webmasterbibliotekerne. Jeg kan naturligvis ikke huske, hvad det var, det var en dialog mellem Rolf og Tue (Ballerup) tror jeg.

Det har hele tiden været sådan for webmasterbiblioteker, også på 2016-2, at admin rollet ikke har rullemenuer (i hvert fald når den er kombineret med local admin), men local admin har dem.

#13 Updated by Anonymous over 2 years ago

@simon i #8: Ja jeg har adgang til secure_permissions, vedhæfter skærmdump

#14 Updated by Simon Holt over 2 years ago

Den dobbelte menu vises altså ikke længere for admin brugeren, hvilket vi ellers havde accepteret at leve med da den kun blev vist for netop admin rollen. Det er ikke noget de langt mere benyttede roller skal leve med.

Derudover kan jeg ikke længere folde undermenuerne ud ved mouse-over ...

@Rolf, mit gæt er på billedet du viser, at du faktisk har dobbelt menu. Du er bare lidt svært at se. Det kan også være derfor menuerne ikke folder ud ved mouse-over: fordi toolbar'en ligger sig ovenover.

Prøv at slå toolbar modulet fra og se om det gør en forskel.

@Tina

Vil du ikke lige prøve at rebuilde permission ved at sørge for at der er sat hak i "Load permissions from code" og tryk "Gem indstillinger". Du burde egentlig være blevet gjort, men bare lige så vi kan udelukke, at det ikke er det der driller.

#15 Updated by Anonymous over 2 years ago

@Rolf: Jeg har prøvet at fjerne rollen "local admin", så min webmaster bruger kun er admininistrator, men det gør ingen forskel. Jeg kan stadig ikke se nogen knapper.

#16 Updated by Anonymous over 2 years ago

@Simon i #14: Efter jeg har fjernet "local admin" rollen, så min bruger kun har "administrators" rollen har jeg fortsat en dobbelt menubar (og ingen genvejsmenu). Men nu kan jeg få lov at se dropdownmenuer i den øverste menu.

#17 Updated by Simon Holt over 2 years ago

@Tina Tænker der må være et eller andet med dine permissions. Vil du ikke lige prøve at rebuild dem?

Er i øvrigt ved at eksperimentere med den ny løsning på alt det her med toolbar/admin_menu, for det skaber godt nok mange problemer.

#18 Updated by Rolf Madsen over 2 years ago

Det er korrekt at jeg har dobbeltmenuen og at den blot dækkes af Toolbar.

#19 Updated by Rolf Madsen over 2 years ago

  • Assignee changed from Gitte Barlach to Simon Holt

#20 Updated by Rolf Madsen over 2 years ago

  • Related to Bug #2929: webmasterpanel forsvinder i “4.0.0-kun-test”, hvis man kommer til at trykke på grå pil added

#21 Updated by Tanja Kate Jensen over 2 years ago

I Hillerød oplever vi også dobbeltmenu, og kan ikke se redigér-knappen i “4.0.0-kun-test”, både når vi er logget ind som admin og som local admin (ved sidstnævnte har vi også rettighederne editor og local editor, som lader til at skabe problemer, så vidt jeg forstå af ovenstående).

#22 Updated by Sofie Fredborg over 2 years ago

Vi har samme problem i Kolding. (Er webredaktør Bibliotek) - både dobbeltmenu og at det dækker over toppen af siden, så man ikke kan komme til at redigere i den - jeg har også prøvet forskellige browsere det gør ingen forskel /Sofie

Tanja Kate Jensen skrev:

I Hillerød oplever vi også dobbeltmenu, og kan ikke se redigér-knappen i “4.0.0-kun-test”, både når vi er logget ind som admin og som local admin (ved sidstnævnte har vi også rettighederne editor og local editor, som lader til at skabe problemer, så vidt jeg forstå af ovenstående).

 

#23 Updated by Christel Krabbenhøft over 2 years ago

Simon, har du mulighed for at kigge mere på denne?

#24 Updated by Simon Holt over 2 years ago

Ja, det vil jeg gerne :)

Er lige i fuld gang med at få #1404 ud af verden, men kaster mig over den så snart jeg er færdig med det.

#25 Updated by Christel Krabbenhøft over 2 years ago

Smukt! Mange tak, Simon :-)

#26 Updated by Katrine Isaksen over 2 years ago

Hej

Ved Norddjurs Biblioteker har vi også problemet med dobbeltmenu, når vi logger ind som ren administrator og kan dermed heller ikke se redigér-knappen!

 

#27 Updated by Simon Holt over 2 years ago

Hej Katrine,

Kan jeg ikke få dig til at skrive https://norddjurs-dev.ddbcms.dk/user/me/roles i adressefeltet og tag et screenshot mens du er logget med den omtalte admin bruger?

 

#28 Updated by Michael W. Christoffersen over 2 years ago

Du kan få et dump fra Albertslund, hvor der mangler noget af listen. Jeg mener, at der er et issue på problemet med menuerne i toppen. Det er i hvert fald et "gammelt" problem.

Albertslunds dev kører på nyeste RC.

#29 Updated by Simon Holt over 2 years ago

Kan se du har user id 1 og det er der problemet er.

Jævnfør Kapsers kommentar her: https://platform.dandigbib.org/issues/2327#note-14

Jeg tænker ikke bibliotekspersonale bør benytte bruger 1 til at administrere deres site eftersom det netop omgår al adgangskontrol i systemet

I løsningen til admin_menu/toolbar problematikken benytter vi os af adgangssystemet til at begrænse hvem der skal have adgang til hvilken bar.

Jeg er dog ved at undersøge om vi kan finde en bedre løsning, men vil bare lige pointere det.

Lige for at opsummere. Dobbet admin bar opstår når:

1. Man benytter admin bruger med id 1.

2. Man benytter en kombination af roller, der giver adgang til begge bars.

 

#30 Updated by Michael W. Christoffersen over 2 years ago

Skrev dette, mens du (@Simon) skrev dit:

Problemet relaterer vist til at det en "forkert" konto der logges på med. @Simon du har været omkring problemet tidligere, hvor du adresserer at man ikke skal bruge 1. bruger - det er vel samme problem her?

I givet fald, skal sagen vel lukkes, og Katrine skal lave en ny admin-bruger?

#31 Updated by Simon Holt over 2 years ago

Kunne godt tænke mig lige at få verificeret, om det også er tilfældet for Katrine at hun bruger id 1.

@Katrine kan du producere det i ovenstående omtalte screenshot?

Synes ikke vi bare skal lukke sagen. Den her problematik vil blive ved med at dukke op igen, fordi bibliotekerne kombinerer rollerne.

Som jeg det skal der tages stilling til følgende:

1. Skal jeg fortsætte med at eksperimentere med en bedre løsning? Som det ser ud nu, vil det medfører en mindre ændring i hvordan nogle moduler anvender tilladelser. Det jeg arbejder hen i mod, er helt at skrotte toolbar, så alle rollerne kan anvende den forbedrede admin_menu. Men må indrømme, at jeg på nuværende tidspunkt ikke er sikker på, om det overhovedet kan lade sig gøre (på en hensigtsmæssig måde). 

2. Skal vi beholde den nuværende løsning og i stedet gennemgå alle rollerne og sørge for, at det ikke burde være nødvendigt at kombinere dem (hvis det da overhovedet kan lade sig gøre)? Hvis vi vælger dette, bør vi som minimum også nævne i manualen, at man kan få dobbelt bar, hvis man kombinerer rollerne og at de er sat op, så det ikke bør være nødvendigt.

#32 Updated by Michael W. Christoffersen over 2 years ago

Har lige kigget på mine to admin-brugere, og eneste forskel er at den ene også er staff - skulle det give problemet med dobbeltmenu?

#33 Updated by Simon Holt over 2 years ago

Nej, det burde ikke give problemer eftersom staff rollen ikke har adgang til noget admin bar.

#34 Updated by Michael W. Christoffersen over 2 years ago

Så vil jeg helt klart mene, at det er løsning 1, som er næste skridt.

#35 Updated by Michael W. Christoffersen over 2 years ago

  • Related to Bug #3020: Rediger knap skjules hvis hoved- og sekundær menu er for lang og sekundær menu skjules hvis admin menu vises og genvejsmenu foldes sammen added

#36 Updated by Lotte Tøstesen over 2 years ago

Rolf har bedt mig supplere, jeg har nogle observationer i https://platform.dandigbib.org/issues/3020#note-8 som I måske kan tage med i loopet, det ser ud til at det er et problem som ikke kun gælder webmaster, jeg har i hvert fald samme problem på et redaktørbibliotek med en almindelig editor og delvist på lokaladmin.

#37 Updated by Simon Holt over 2 years ago

Så er jeg klar med et PR med nyt forslag til løsning, som forhåbentlig kan få os ud af alt den her suppedas.

Kort forklaring: admin_menu og toolbar benytter Drupals interne menu API til at bygge toolbar/admin bar. Det der bliver vist afhænger dermed af den adgangskontrol, der bliver udført i menu API. 

Tilladelsen "access administration pages" bruges til at give adgang til overordnede menupunkter (se bl.a. hook_menu implementation i system module). Man kan ikke bare give adgang til alle på disse, så man er nødt til at have en tilladelse til at styre dette. Det betyder så, at denne tilladelse helst ikke må bruges til andet end det og at alle roller, der skal bruge en admin bar som f.eks. admin_menu, skal have denne tilladelse.

Vi ville ikke bare give denne tilladelse til alle roller i DDB CMS, da den så ud til at være blevet misbrugt flere steder.

Efter at have gravet lidt i det, var det faktisk slet ikke så slemt. De fleste stedet blev den brugt korrekt som f.eks. i Ting modulet, hvor den bliver brugt til at give adgang til den overordnede fane for Ting-indstillinger. Der var kun et sted den blev misbrugt og det har jeg rettet i nedenstående PR.

Har også eksporteret indstillinger for admin_menu med strongarm. Før blev det indstillet i en update, men det havde ikke nogen effekt i en frisk installation.

PR: https://github.com/ding2/ding2/pull/877

En sidenote: Jeg rebasede mit PR på master og kunne efterfølgende slet ikke få vist forsiden. Prøvede også en frisk installation og den stod bare og loadede på forsiden, men der skete aldrig noget. Var faktisk nødt til at dræbe Chrome-processen for at kunne lukke browser-vinduet. Andre der har oplevet dette? 

#38 Updated by Rolf Madsen over 2 years ago

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

#39 Updated by Simon Holt over 2 years ago

Noter til test:

1. Opret en bruger for hver af rollerne, der har fået 'access administration pages' tildelt:

- editor
- guest_blogger
- local_editor
- user_administrator

Test at de ikke har adgang til noget, de ikke skulle have adgang til.

2. Opret en bruger med local_administrator og editor rolle. Test at der ikke forekommer dobbelt admin bar.

3. Test at der ikke kommer dobbelt admin bar med bruger id 1.

 

#40 Updated by Christel Krabbenhøft over 2 years ago

  • Target version changed from DDB CMS 2017 1.1 opgradering (B14 sprintbacklog) to Release 27 - Bugfixes (B14)

#41 Updated by Katrine Isaksen over 2 years ago

Hej

Jeg kan slet ikke følge med i denne tråd og er derfor ikke klar over, om problemet er løst! 

Jeg har forsøgt at lave ovennævnte screenshot...

 

 

#42 Updated by Michael W. Christoffersen over 2 years ago

@Katrine, sagen forventes løst i næste release, men tak for screenshot.

#43 Updated by Kasper Garnæs about 2 years ago

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

Reviewed og koden er godkendt. PR'et skal opdateres for at kunne merges.

Bemærk at den foreslåede løsning slår Drupals Toolbar modul helt fra så at vi fremadrettet kun bruger ét modul til administrationsmenuen. Det vil medføre at den ser (lidt) anderledes ud for de brugere, der tidligere kun har været vandt til Toolbar.

#44 Updated by Simon Holt about 2 years ago

Jeg har rebased PR.

#45 Updated by Simon Holt about 2 years ago

  • Assignee changed from Simon Holt to Rolf Madsen

Troede egentlig denne var blevet merget, men så lige der stadig var dobbelt admin bar efter installation af nyeste kode.

@Rolf håber det er OK jeg assigner den til dig, så du kan finde ud af hvad der skal med den.

Kasper reviewede PR sidst og jeg har opdateret PR, så den kan merges.

#46 Updated by Rolf Madsen about 2 years ago

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

Tak for heads-up Simon!

PR mangler merge til Ding2.

#47 Updated by Gitte Barlach about 2 years ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs

#48 Updated by Rolf Madsen about 2 years ago

  • Priority changed from Normal to High

#49 Updated by Kasper Garnæs about 2 years ago

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

Merged.

#50 Updated by Michael W. Christoffersen about 2 years ago

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

Testet og godkendt.

#51 Updated by Simon Holt about 2 years ago

  • Related to Bug #2184: Toolbar (menu) virker ikke for local editor og guest blogger added

Also available in: Atom PDF