Nette GeoIP extension – Nějaký nette guru zde?

Isigarek
Člen | 74
+
0
-

Dobrý den,
jelikož jsem začátečník píšu jsem ohledně toho aby nějaký guru zkontroloval popřípadě mi nahlásil možné upravy chyby nebo tak podobně z mého prvního extension pro nette.

Jedná o jednoduché API pro GeoIP, které bere data z https://freegeoip.net/ .
Jelikož bych se moc rád zlepšil tak bych byl taktéž velice rád za to, kdyby si někdo našel ten čas a podíval se na to. Díky!

PS: Jak jsem tam psal v doc. panel diagnostiky je použít hlavně z Kdyby/Facebook jelikož takový panel s časem a rozklikáváním sem dělal poprvé tak doufám, že to nevadí :)

GITHUB: https://github.com/…/nette-geoip

GEpic
Člen | 566
+
+1
-

Ahoj, proč to má závislosti na balíčcích jako je nette/mail, nette/database atp?

PS: Geolokace podle IP je asi tak přesná jako věštecká koule (já jsem podle všeho z Karlovarského kraje) – dnes už je větší šance na správné zaměření uživatele podle GPS lokace v mob. zařízeních.

Editoval GEpic (20. 1. 2018 12:00)

Isigarek
Člen | 74
+
0
-

To sice ano. Jaké tedy by měli být requirements? Určitě DI, a dal? Database, mail oddelam díky

ali
Člen | 342
+
+1
-

Zavislosti musi byt na tech baliccich, ktere ta komponenta potrebuje.

CZechBoY
Člen | 3608
+
+2
-

@GEpic Pokud staci presnost na zaklade zeme tak to staci ;-)

Isigarek
Člen | 74
+
0
-

To taktéž chápu. Spíš mi jde o to jestli stačí pro nette extension jenom DI nebo ještě něco třeba nette/application

ali napsal(a):

Zavislosti musi byt na tech baliccich, ktere ta komponenta potrebuje.

Isigarek
Člen | 74
+
0
-
	"nette/bootstrap": "^2.4.3", - $compiler extension?
"nette/di": "^2.4", - DI pro extend extension
"nette/utils": "^2.4", - ?
"latte/latte": "^2.4", - Využívá tracy panel
"tracy/tracy": "^2.4", - Tracy bar?

Je to takto v pořádku?

Editoval Isigarek (20. 1. 2018 13:23)

GEpic
Člen | 566
+
0
-

@GEpic Pokud staci presnost na zaklade zeme tak to staci ;-)

Na to jsem nepomyslel, díky!

Isigarek napsal(a):

	"nette/bootstrap": "^2.4.3", - $compiler extension?
"nette/di": "^2.4", - DI pro extend extension
"nette/utils": "^2.4", - ?
"latte/latte": "^2.4", - Využívá tracy panel
"tracy/tracy": "^2.4", - Tracy bar?

@Isigarek No smazal si si akorát nette/caching které tam ale využíváš. Vyzkoušej si ten balíček vždy nainstalovat do nového projektu ve kterém není celé nette/nette.

Editoval GEpic (20. 1. 2018 16:20)

Isigarek
Člen | 74
+
0
-

Právě že ne využívám protože guzzle cachuje sám. Možná je ještě v constructoru když tak ho výmazu

GEpic
Člen | 566
+
+1
-

Ještě otázka jestli tracy/tracy nepřesunout pouze do require-dev.

EDIT: use statement

Editoval GEpic (20. 1. 2018 16:21)

Isigarek
Člen | 74
+
+1
-

To je dobrá poznámka tím pádem by tam šlo i latte je využito pouze k Tracy panelu

GEpic
Člen | 566
+
+1
-

@Isigarek a když budu fakt fest hnidopich mrkni se na nette coding standard s odřádkováním viz třeba zde