Bug #337

Link zu einem Logeintrag tlws. ohne Cache-ID

Added by Schrottie almost 6 years ago. Updated over 5 years ago.

Status:erledigt% Done:

100%

Priority name:1 niedrig
Assignee:bohrsty
Target version:Version 11
Ticket Referenz: Kategorien:

Description

siehe auch: http://forum.opencaching-network.org/index.php?topic=3579.msg45817#msg45817

In einigen Fällen wird das Logtypicon eines Logeintrages mit einem fehlerhaften Permalink zum Logeintrag hinterlegt. Dies tritt nachvollziehbar immer dann auf, wenn das Javascript ausgelöst wurde, das beim Scrollen an das Ende des Listings weitere Logeinträge nachlädt. In diesem Fall werden die Links ohne Cache-ID angezeigt. Auch bei bis dahin korrekten Links tritt dieser Fehler dann auf.


Related issues

Duplicated by OC Entwicklung - Bug #359: Verlinkungsfehler erledigt

Associated revisions

Revision 77c32c0e
Added by bohrsty over 5 years ago

added alias for "cacheid" in cache sql command, updates #337

Revision 58b735eb
Added by bohrsty over 5 years ago

Merge pull request #142 from bohrsty/337_loglink_bug

added alias for "cacheid" in cache sql command, updates #337

History

#1 Updated by Schrottie almost 6 years ago

Beteiligt an den Logs sind folgende Templates aus /htdocs/template2/ocstyle:

res_logentry.tpl - Erzeugt den Logbereich, wird inkl. Übergabe der Cache-ID von viewcache.tpl bzw. viewlogs.tpl aufgerufen.
res_logentry_logitem.tpl - Erzeugt den jeweiligen Logeintrag und auch damit den Link. Wird von res_logentry.tpl aufgerufen und bekommt die Cache-ID übergeben.

Das Javascript, das für das Nachladen der Logeinträge verantwortlich ist, befindet sich in viewcache.tpl. Das Javascript ruft dabei viewlogs.php (und nutzt damit viewlogs.tpl) auf. Dabei wird innerhalb von viewlogs.php die Funktion(?) 'tagloadlogs' verwendet, die für die anzeige der kompletten Logliste sorgt:

    // 'tagloadlogs' produces a stripped-down version of the loglist for
    // log autoloading (see viewcache.php). The actual log block to be inserted
    // is tagged with <ocloadlogs>...</ocloadlogs>.

Hier wird vermutlich die Cache-ID unterschlagen.

#2 Updated by Schrottie almost 6 years ago

  • Priority name changed from 2 mittel to 1 niedrig
  • Target version set to Version 11

#3 Updated by bohrsty over 5 years ago

viewcache.php and viewlogs.php uses different sql commands to fill the $cache array in templates... in viewlogs.php $cache.cacheid is unknown and empty, because the sql command doesn't aliases `caches`.`cache_id`...

solution:
add:

`caches`.`cache_id` AS `cacheid`,

in viewlogs.php at the end of line 55:
(see viewcache.php line 88)...

#4 Updated by bohrsty over 5 years ago

  • Status changed from neu to offen
  • Assignee set to bohrsty

#5 Updated by bohrsty over 5 years ago

  • Status changed from offen to erledigt
  • % Done changed from 0 to 100

fixed by adding the mentioned sql alias...

Also available in: Atom PDF