migol / opencaching-pl

Automatically exported from code.google.com/p/opencaching-pl
0 stars 0 forks source link

Wprowadzenie możliwości dodania kilku (2, 3...n ) współrzędnych bazowych #32

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Dotyczy wątku:
http://forum.opencaching.pl/viewtopic.php?f=9&t=4802&p=117813#p117802

Prośba o "studium wykonalności" - czy jest to możliwe, czy bardzo 
pracochłonne, czy nie wprowadzi zamieszania, przeciążenia serwera i jakichś 
tragicznych następstw.

Z założenia każdy użytkownik miałby możliwość zadeklarowania jednych 
współrzędnych domowych (podstawowych) i kilku innych bazowych (konieczne 
ustalenie ile jest sensowne/możliwe) i promienia w jakim otrzymywałby z tych 
kilku miejsc powiadomienia.

Jeśli niemożliwe - proszę o zamknięcie.

Original issue reported on code.google.com by etnam...@gmail.com on 3 Feb 2014 at 9:31

GoogleCodeExporter commented 9 years ago
Jak najbardziej możliwe, w podstawowej wersji nawet nie takie trudne. Dla 
kogoś kto jest obeznany w kodzie OC to jest kilka godzin pracy, dla 
pozostałych kilkanaście.

Podstawowa wersja, czyli to co jest zgłaszane w wątku:

- możliwość dodania innych współrzędnych jako 'domowych' w panelu 
użytkownika i ich wyświetlenie w tymże panelu.
- po kliknięciu na 'moja okolica pokazuje się okolica domyślna, dodatkowo 
pojawia się menu (selektor?) z wszystkimi dodatkowymi współrzędnymi 
wprowadzonymi przez użytkownika. (jeśli user jakieś dodał)
- po wybraniu innych współrzędnych ajaxami dociągne są dane z innych 
współrzędnych i podmieniane w widoku strony.

Od strony silnika sugeruję:
- globalny wyłącznik (boolean) funkcjonalności w lib/settings.inc.php
- wszystkie operacje silnikowe (nie związane z wyświetlaniem warstwy 
graficznej) związane z obsługą tej funkcjonalności zawrzeć w odrębnej 
klasie, obecny kod rozdzielić: ten który jest związany z warstwą 
pozostawić a algorytmy przenieść do tejże klasy (może być kilka klas w 
miarę potrzeb)

Podstawową rzeczą, której pewnie osoby piszące w wątku nie mają 
świadomości, że dodanie kolejenych współrzędnych nie wiąże się z 
wysyłaniem emaili powiadamoających o nowych skrzynkach itp. 
Ta funkcjonalność wymagała by głębszych zmian w silniku, w tej chwili nie 
wiem jeszcze jak dużo trzba by zmieniać. jeśli kiedyś było to napisne 'po 
bożemu', to też niewiele, jednak jeśli jest napisane kiepsko, to będzie 
więcej roboty. (nigdy w tamtą częśc kodu nie zaglądałem, a w kodzie 
niektóre rzeczy są napisane bardzo fajnie a inne bardzo nie fajnie, więc 
trudno wróżyć z fusów jak jest napisany tamten moduł)

No i teraz brakuje tylko kogoś, kto się podejmie :-)
jeśli dzazder się podejmie, to mogę pomóc (tzn. wskazać drogę, gdzie co 
jest i jak to zrobić, na rozgrzebywanie kodu nie będę mieć czasu - mam 
jeszcze parę issues do zrobienia i chciałbym się na nich skupić.)

Original comment by wloczynutka on 3 Feb 2014 at 5:49

GoogleCodeExporter commented 9 years ago
Witam serdecznie,
mogę się podjąć implementacji tegoż rozwiązania, pytanie tylko czy jako 
gość, który w kodzie nie za wiele grzebał, nie namieszam tam za bardzo i 
nie zrobię bałaganu :)

Pozdrawiam
dżazder

Original comment by dzaz...@gmail.com on 4 Feb 2014 at 10:05

GoogleCodeExporter commented 9 years ago
jeśli weźmiesz pod uwage to co pisalem powyzej, a zwlaszcza:
- globalny wyłącznik (boolean) funkcjonalności w lib/settings.inc.php
- zmiany zawrzeć w odrębnej klasie (...)
to powinno byc ok, bo:
- na poczatku bedzie mozna modul wlaczyc tylko dla wybranych uzytkownikow - 
testerow, dzieki temu bedziesz mogl sobie podzielic prace i commitowac 
poszczegolne etapy, a wybrani userzy beda mogli je stestowac, 
- zawsze bedzie mozna wylaczyc dla calego portalu w razie czego.
- na poczatek mozesz też np. stworzyc jakiś plik myneighborhood_v2.php, i do 
niego klase silnika np. lib/myNeighborhood.php a jak skończysz prace i 
wszystko bedzie stestowane myneighborhood.php można bdize zastąpic 
myneighborhood_v2.php

Original comment by wloczynutka on 4 Feb 2014 at 4:06

GoogleCodeExporter commented 9 years ago

Original comment by wloczynutka on 31 Dec 2014 at 12:17