Bug #870

Bei der Umkreissuche von "suchbare"n Caches um einen Cache werden auch deaktivierte Caches angezeigt

Added by kirstenko over 3 years ago. Updated over 3 years ago.

Status:erledigt% Done:

100%

Priority name:1 niedrigEstimated time:1.00 h
Assignee:kirstenko
Target version:Version 16
Ticket Referenz: Kategorien:suche

Description

Am Beispiel:
Auf der Seite eines Caches liefert der Link "suchbare" (Caches im Umkreis: alle - suchbare - gleiche Cacheart) auch nicht verfügbare Caches (nämlich in meinem Fall bei der Suche ausgehend vom Cache http://opencaching.de/OC12440 den Treffer http://opencaching.de/OC11674).

Associated revisions

Revision 16efca06
Added by kirstenko over 3 years ago

don't show any disabled caches, when searching for nearby searchable or same type geocaches; updates #870

Revision d606febd
Added by mambofive over 3 years ago

Merge pull request #231 from kirstenko/870_nearby_searchable_and_same_type_geocaches

updates #870 don't show any disabled caches, when searching for nearby searchable or same type geocaches …

History

#1 Updated by kirstenko over 3 years ago

Ich würde den Bug gerne auch bearbeiten, habe aber noch keine Möglichkeit gefunden, ihn in Bearbeitung zu nehmen bzw. mir zuzuweisen.
Gruß Jan

#2 Updated by kirstenko over 3 years ago

In der Entwicklungsumgebung erfüllen die folgenden Caches die gleichen Bedingungen, um den Fall zu testen:
Aktiver Cache OC8509 "Loch".
Suche im Umkreis "suchbare" liefert auch den folgenden, momentan nicht verfügbaren, Cache OC2E73 "Studienseminar Gedenkenscache" (bei der Wegpunkt-Suche muss man ggf. den Port 1080 in der URL ergänzen).

#3 Updated by kirstenko over 3 years ago

Die Änderung an sich wird vermutlich nur ganz wenig Zeit in Anspruch nehmen. Was mich nun schon für längere Zeit beschäftigt hat, ist die Suche nach der richtigen Quelldatei. Zunächst war ich fälschlich unter htdocs\cache2\smarty\compiled gelandet. Dort liegen aber generierte Dateien, bei denen eine Änderung nicht viel Sinn macht.
Die tatsächlich (hoffentlich) richtige Stelle ist htdocs\templates2\ocstyle\viewcache.tpl Zeilen 506-508 für "alle - suchbare - gleiche Cacheart":

...&f_userowner=0&f_userfound=0&f_inactive=1...
...&f_userowner=1&f_userfound=1&f_inactive=1...
...&f_userowner=1&f_userfound=1&f_inactive=1...&cachetype={$cache.type}...

Hier müsste man sich evtl. noch einmal Gedanken machen, was man denn tatsächlich anzeigen möchte.
Ich habe in meiner Entwicklungsumgebung ein wenig herumexperimentiert. Es gibt Möglichkeiten zum anzeigen/ausblenden von
- archivierten Caches (f_inactive=0/1)
- momentan nicht verfügbaren Caches (f_disabled=0/1)
- ignorierten Caches (f_ignored=0/1)

Meiner Meinung nach macht es Sinn, wenn man zumindest für "suchbare" und "gleiche Cacheart" keine archivierten und keine momentan nicht verfügbaren Caches ausgibt (weil sie eben nicht suchbar sind). Aber auch ignorierte Caches möchte hier vermutlich niemand angezeigt bekommen.

Bei allen Anzeigen werden bisher schon Caches mit den Stati "gesperrt" und "gesperrt,versteckt" nicht angezeigt.

#4 Updated by kirstenko over 3 years ago

  • Status changed from neu to in Arbeit 80%
  • Assignee set to kirstenko
  • % Done changed from 0 to 80

#5 Updated by following over 3 years ago

kirstenko schrieb:

Meiner Meinung nach macht es Sinn, wenn man zumindest für "suchbare" und "gleiche Cacheart" keine archivierten und keine momentan nicht verfügbaren Caches ausgibt (weil sie eben nicht suchbar sind). Aber auch ignorierte Caches möchte hier vermutlich niemand angezeigt bekommen.

Finde ich auch sinnvoll so.

#6 Updated by kirstenko over 3 years ago

  • Status changed from in Arbeit 80% to erledigt
  • % Done changed from 80 to 100

Es wurde der Parameter f_disabled=1 für die Suchen "suchbare" und "gleiche Cacheart" hinzugefügt, so dass nun keine Caches mit Status "momentan nicht verfügbar" mehr angezeigt werden. Ignorierte Caches wurden bereits nicht angezeigt, so dass keine Notwendigkeit bestand, auch den Parameter f_ignored zu besetzen.

#7 Updated by following over 3 years ago

  • Status changed from erledigt to in Arbeit
  • Priority name changed from 0 keine to 1 niedrig
  • % Done changed from 100 to 0
  • Kategorien set to suche

ist noch nicht im Master-Branch angekommen

#8 Updated by kirstenko over 3 years ago

Hallo zusammen.
Heute konnte ich, nachdem alle vorherigen (Fehl-)versuche rückgängig gemacht wurden, die Änderung vornehmen und per push zum github.com schieben. Dort habe ich einen pull request gestartet, der ziemlich fix von Elmar gemerged wurde. Damit dürfte der Vorgang komplett beendet sein, oder fehlt noch etwas?
(Meinen neu erstellten branch "870_nearby_searchable_and_same_type_geocaches" kann ich nun vermutlich lokal und im origin löschen.)
Viele Grüße
Jan

#9 Updated by following over 3 years ago

  • Status changed from in Arbeit to im Test
  • Target version set to Version 16
  • % Done changed from 0 to 90

ok, danke!

#10 Updated by following over 3 years ago

  • Status changed from im Test to erledigt
  • % Done changed from 90 to 100

Also available in: Atom PDF