Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#61 closed task (fixed)

Littered URLs

Reported by: wschmidle Owned by: abukhman
Priority: major Milestone:
Component: echo-textdisplay Version:
Keywords: discuss Cc:
Parent Tickets: #174

Description

URLs sollten nur die Parameter enthalten, die tatsächlich relevant sind.

Beispiel: Wenn man mit der "Start-URL" für Benedetti anfängt:
http://mpdl-dev.mpiwg-berlin.mpg.de/ECHOdocuView?url=/mpiwg/online/permanent/library/163127KK
und dort auf Seite 398 geht, sollte die URL zu
http://mpdl-dev.mpiwg-berlin.mpg.de/ECHOdocuView?url=/mpiwg/online/permanent/library/163127KK&pn=398
werden. Tatsächlich wird die URL zu
http://mpdl-dev.mpiwg-berlin.mpg.de/ECHOdocuView?url=/mpiwg/online/permanent/library/163127KK&viewMode=text_dict&tocMode=thumbs&tocPN=1&query=&searchPN=1&queryType=&mode=texttool&sn=&queryPageSize=10&pn=398.
Dinge wie "&sn=" sind überflüssig.

Vermutlich sollte die URL auch keine default-Einstellungen enthalten, solange der Benutzer sie nicht aktiv ausgewählt hat, d.h. auch Dinge wie "&viewMode=text_dict" sind überflüssig. (Das ändert sich natürlich, wenn der Benutzer zum Beispiel von "Text+Dictionary" auf "Text" und wieder zurück auf den default "Text+Dictionary" klickt.) Es gibt jedenfalls keinen Grund, die default-Einstellungen in der "Start-URL" wegzulassen und nach der ersten User-Interaktion nachzutragen.

Subtickets

Change History (11)

comment:1 Changed 9 years ago by wschmidle

Vielleicht klarer:

... (Das ändert sich natürlich, wenn der Benutzer zum Beispiel von "Text+Dictionary" auf "Text" und wieder zurück auf "Text+Dictionary" klickt, weil es dann nicht mehr die default-Anzeige, sondern eine bewusste Wahl des Users ist.) ...

comment:2 Changed 9 years ago by abukhman

  • Resolution set to invalid
  • Status changed from new to closed

comment:3 Changed 9 years ago by abukhman

Ich kann nicht URL ändern

comment:4 Changed 9 years ago by wschmidle

  • Resolution invalid deleted
  • Status changed from closed to reopened
  • Type changed from defect to task

mit Andrey besprochen: sehr schwierig, kann nur langfristig geändert werden

comment:5 Changed 9 years ago by abukhman

Ich denke es ist überhaupt nicht nötig url zu ändern.

comment:6 Changed 8 years ago by abukhman

  • Resolution set to fixed
  • Status changed from reopened to closed

Ich habe in allen möglichen Stellen den unbenutzte Linkteile gelöscht.

comment:7 Changed 8 years ago by wschmidle

  • Resolution fixed deleted
  • Status changed from closed to reopened

Entschuldige, Andrey, aber ich kann nicht sehen, was du verändert hast. Das, was ich im Ticket beschreibe, ist jedenfalls immer noch da. Es ist sogar (neben "&characterNormalization=reg") noch die leere Option "&optionToggle=" dazugekommen.

Wenn der Benutzer die "Start-URL" aufruft, wäre es dann einfacher, doch automatisch alle relevanten Optionen explizit and die Start-URL anzuhängen? Aber bitte keine leeren Optionen wie "&optionToggle=".

Beispiel: Das System wandelt

http://mpdl-dev.mpiwg-berlin.mpg.de/ECHOdocuView?url=/mpiwg/online/permanent/library/163127KK

automatisch um in

http://mpdl-dev.mpiwg-berlin.mpg.de/ECHOdocuView?url=/mpiwg/online/permanent/library/163127KK&pn=5&viewMode=text_dict&characterNormalization=reg&tocMode=thumbs&tocPN=1&optionToggle=0

(pn=5, weil das im Benedetti die Titelseite ist.)

Was bewirkt "&mode=texttool"?

Was bewirkt "&queryPageSize=10"?

Und kann man in der URL "/" statt "%2F" verwenden?

comment:8 Changed 8 years ago by wschmidle

Aber eigentlich finde ich es sinnvoller, wenn sich das System bei der URL

http://mpdl-dev.mpiwg-berlin.mpg.de/ECHOdocuView?url=/mpiwg/online/permanent/library/163127KK

die default-Optionen

&pn=5&viewMode=text_dict&characterNormalization=reg&tocMode=thumbs&tocPN=1&optionToggle=0

dazudenkt, ohne sie explizit der URL anzuhängen.

comment:9 Changed 8 years ago by wschmidle

  • Keywords discuss added

comment:10 Changed 8 years ago by abukhman

  • Resolution set to fixed
  • Status changed from reopened to closed

Alle Elemente sind wichtig den Rest habe ich gelöscht.

comment:11 Changed 8 years ago by wschmidle

  • Parent Tickets set to 174
Note: See TracTickets for help on using tickets.