search

kir-dev / korok

Villanykari Profil és Körök alkalmazás
MIT License
6 stars 0 forks source link

Tárolt eljárások és triggerek eltávolítása #92

Closed tmichel closed 10 years ago

tmichel commented 10 years ago

Főleg a #67-es issue lezárása.

Az adatbázis séma módosításához készült egy rövid kis ruby script. Ez legenerálja az eddigi pontokat beszúró sql szkriptet.

Ezen túl még a Configuration osztály néhány hibáját is javítja ez a PR.

vbalazs commented 10 years ago

:clap: LGTM :shipit:

vbalazs commented 10 years ago

ping @Ksisu do you have time to review this PR?

messo commented 10 years ago

Ha az egyszerűsítés a cél akkor lehetne áttérni no-interface view-s EJB-kre.

On Sun, May 4, 2014 at 12:57 PM, Balazs Varga notifications@github.comwrote:

ping @Ksisu https://github.com/Ksisu do you have time to review this PR?

— Reply to this email directly or view it on GitHubhttps://github.com/kir-dev/korok/pull/92#issuecomment-42130167 .

Üdv, Kriván Bálint

vbalazs commented 10 years ago

@messo az a baj, hogy akkor az ejb implementacios package-t kell betenni a web retegnek fuggosegkent. Mar ha egyre gondolunk. Az meg egy dolog, hogy nem elegans, de lehet problemakat is okozhat. De ha neked van ebben pozitiv tapasztalatod, akkor meg lehet probalni.

messo commented 10 years ago

Sztem no-interface view-nál nem úszod meg hogy függőség legyen, ez sztem nem elegancia kérdése, de javítson ki valaki. Persze ha csak az interfészekre függesz rá, akkor csak azokat éred el, és akkor ha teszel publikus metódust az implementációba akkor sincs gond mert nem látod, de ha valaki hackelni akar akkor felvesz az interfészbe egy új metódust és tádá. Az egész interfészes játék akkor érdekes ha külön akarod választani a Local és Remote finomságokat, de mi esetünkben ez nincs, tehát jóság van (mondjuk régi időkből rémlik valami UserManagerBean Remote-tal de lehet csak álmodtam, és ez talán akkor volt amikor a profil is külön alkalamzás volt).

2014-05-04 15:23 GMT+02:00 Balazs Varga notifications@github.com:

@messo https://github.com/messo az a baj, hogy akkor az ejb implementacios package-t kell betenni a web retegnek fuggosegkent. Mar ha egyre gondolunk. Az meg egy dolog, hogy nem elegans, de lehet problemakat is okozhat. De ha neked van ebben pozitiv tapasztalatod, akkor meg lehet probalni.

— Reply to this email directly or view it on GitHubhttps://github.com/kir-dev/korok/pull/92#issuecomment-42132838 .

Üdv, Kriván Bálint

tmichel commented 10 years ago

@messo abban igazad van, hogy csak local view van és remote nincs, így lehetne interfész nélkül is, de én továbbra is rejteném az implementációt. mindig lehet bele extra metódusokat tenni, nem az a cél, hogy ne lehessen (mondjuk az nem baj ha egy picit nagyobb effort, mert akkor nem szemetelik tele).

szerintem egy jó mentális szeparációt is biztosít az, hogy ha van egy interfész réteg és csak azon keresztül érhető el az ejb réteg és nincs közvetlen hozzáférés

messo commented 10 years ago

Tudtam, hogy valaki fel fogja hozni a szeparációt, mert az egész "csak interfészt publikáljunk" dolognak ez a lényege. Két egymással beszélő komponensnél ez kurva fasza (nehogy mán az internal stuffot hívja egy random caller), de sztem egy üzleti logikát biztosító rétegnél fölösleges. Ez a nagyobb effort, csak felbassza a fejlesztőt, az meg hogy nem szemetelik tele: review, revert stb :)

Még egy apróság, hogy a build idő rövidülne, persze ez számít a legkevésbé, kevesebb projekt, talán egy bolha f*sznyival átláthatóbb ;)

Persze ez mind csak az én távoli beleszólásom a projekt irányába, de néha jó kiélni az eféle perverzióim ^^

2014-05-04 19:37 GMT+02:00 Tamás Michelberger notifications@github.com:

@messo https://github.com/messo abban igazad van, hogy csak local view van és remote nincs, így lehetne interfész nélkül is, de én továbbra is rejteném az implementációt. mindig lehet bele extra metódusokat tenni, nem az a cél, hogy ne lehessen (mondjuk az nem baj ha egy picit nagyobb effort, mert akkor nem szemetelik tele).

szerintem egy jó mentális szeparációt is biztosít az, hogy ha van egy interfész réteg és csak azon keresztül érhető el az ejb réteg és nincs közvetlen hozzáférés

— Reply to this email directly or view it on GitHubhttps://github.com/kir-dev/korok/pull/92#issuecomment-42138963 .

Üdv, Kriván Bálint

vbalazs commented 10 years ago

Persze ez mind csak az én távoli beleszólásom a projekt irányába, de néha jó kiélni az eféle perverzióim ^^

ez egyaltalan nem baj :) en orulok neki. en is mar csak viszonylag tavolrol szolok bele a dolgokba :sunglasses:

tmichel commented 10 years ago

Szerintem csak az bassza fel magát, aki gányolni szeretne és quick & dirty megoldásokkal akarja megúszni. Egy jól karbantartható kód szerintem nem feltétlen vonja magával azt, hogy minden kézre áll. Sőt általában ezt fel kell áldozni a modularitásért. Viszont kezdünk nagyon elmenni a PR témájától. Én szívesen folytatom ezt, de akkor vigyük a listára a beszélgetést.