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

Merged
merged 15 commits into from May 4, 2014

Conversation

Projects
None yet
3 participants
@tmichel
Contributor

tmichel commented May 4, 2014

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

  • megszűntek a triggerek
  • eltűntek a tárolt eljárások és az egyedi típusok az adatbázisból
  • a közösségi pontokat generáljuk és nem on-the-fly állítjuk elő
  • a közösségi pontokat az értékelési időszak lezárásakor generáljuk

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.

tmichel added some commits Jan 1, 2014

remove trigger for updating user records (#67)
when a spot image is deleted the user's showRecommendedPhoto flag must be set to false
remove trigger for inserting spot images (#67)
spot images are imported, not created by the system
added EntrantExporter (#67)
it will replace the export_entrant_requests stored procedure
Configuration refactor
using CDI for injection
Configuration interface and its implementation ConfigurationImpl

new ejb method: UserManagerBean#removeProfileImage
removed implementation class from sch-pek-ejb-services module
Added PointHistory entity
This stores the community points for a given user in a semester.
point history script
this script generates an sql file. it creates an insert for every user
and every semester.
@vbalazs

This comment has been minimized.

Show comment
Hide comment
@vbalazs

vbalazs May 4, 2014

Contributor

👏
LGTM :shipit:

Contributor

vbalazs commented May 4, 2014

👏
LGTM :shipit:

@vbalazs

This comment has been minimized.

Show comment
Hide comment
@vbalazs

vbalazs May 4, 2014

Contributor

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

Contributor

vbalazs commented May 4, 2014

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

@messo

This comment has been minimized.

Show comment
Hide comment
@messo

messo May 4, 2014

Contributor

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

Contributor

messo commented May 4, 2014

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

This comment has been minimized.

Show comment
Hide comment
@vbalazs

vbalazs May 4, 2014

Contributor

@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.

Contributor

vbalazs commented May 4, 2014

@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.

@vbalazs vbalazs added the refaktor label May 4, 2014

@messo

This comment has been minimized.

Show comment
Hide comment
@messo

messo May 4, 2014

Contributor

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

Contributor

messo commented May 4, 2014

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

This comment has been minimized.

Show comment
Hide comment
@tmichel

tmichel May 4, 2014

Contributor

@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

Contributor

tmichel commented May 4, 2014

@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

This comment has been minimized.

Show comment
Hide comment
@messo

messo May 4, 2014

Contributor

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

Contributor

messo commented May 4, 2014

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

This comment has been minimized.

Show comment
Hide comment
@vbalazs

vbalazs May 4, 2014

Contributor

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 😎

Contributor

vbalazs commented May 4, 2014

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 😎

@tmichel

This comment has been minimized.

Show comment
Hide comment
@tmichel

tmichel May 4, 2014

Contributor

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.

Contributor

tmichel commented May 4, 2014

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.

@tmichel tmichel merged commit 9c5ee4a into master May 4, 2014

@tmichel tmichel deleted the remove-triggers branch May 4, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment