diff options
author | Simon Rettberg | 2015-07-03 16:43:38 +0200 |
---|---|---|
committer | Simon Rettberg | 2015-07-03 16:43:38 +0200 |
commit | 90c159d622f234b99e47293f3ff2a35d64e6f8c9 (patch) | |
tree | e967412b1464e9e3cad381399d5dd1c596c0bf28 /dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/thrift/ServerHandler.java | |
parent | [server] Refactored permission checking classes a bit (diff) | |
download | tutor-module-90c159d622f234b99e47293f3ff2a35d64e6f8c9.tar.gz tutor-module-90c159d622f234b99e47293f3ff2a35d64e6f8c9.tar.xz tutor-module-90c159d622f234b99e47293f3ff2a35d64e6f8c9.zip |
[server] Adapt to API changes, refine permission checking for image version deletion
Diffstat (limited to 'dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/thrift/ServerHandler.java')
-rw-r--r-- | dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/thrift/ServerHandler.java | 35 |
1 files changed, 23 insertions, 12 deletions
diff --git a/dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/thrift/ServerHandler.java b/dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/thrift/ServerHandler.java index 5cf28713..03bb8444 100644 --- a/dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/thrift/ServerHandler.java +++ b/dozentenmodulserver/src/main/java/org/openslx/bwlp/sat/thrift/ServerHandler.java @@ -7,6 +7,7 @@ import java.util.Map; import org.apache.log4j.Logger; import org.apache.thrift.TException; +import org.openslx.bwlp.sat.database.Paginator; import org.openslx.bwlp.sat.database.mappers.DbImage; import org.openslx.bwlp.sat.database.mappers.DbImagePermissions; import org.openslx.bwlp.sat.database.mappers.DbLecture; @@ -56,6 +57,11 @@ public class ServerHandler implements SatelliteServer.Iface { return Version.VERSION; } + @Override + public int getPageSize() throws TException { + return Paginator.PER_PAGE; + } + /* * File Transfer */ @@ -324,22 +330,27 @@ public class ServerHandler implements SatelliteServer.Iface { @Override public LectureRead getLectureDetails(String userToken, String lectureId) throws TAuthorizationException, - TNotFoundException { - // TODO Auto-generated method stub - return null; - } - - @Override - public List<LectureSummary> getLecturesByImageVersion(String userToken, String imageVersionId) - throws TAuthorizationException, TNotFoundException { - // TODO Auto-generated method stub - return null; + TNotFoundException, TInternalServerError { + UserInfo user = SessionManager.getOrFail(userToken); + User.canSeeLectureDetailsOrFail(user); + try { + return DbLecture.getLectureDetails(user, lectureId); + } catch (SQLException e) { + throw new TInternalServerError(); + } } @Override public void deleteLecture(String userToken, String lectureId) throws TAuthorizationException, - TNotFoundException { - // TODO Auto-generated method stub + TNotFoundException, TInternalServerError { + UserInfo user = SessionManager.getOrFail(userToken); + User.canDeleteLectureOrFail(user, lectureId); + try { + if (!DbLecture.delete(lectureId)) + throw new TNotFoundException(); + } catch (SQLException e) { + throw new TInternalServerError(); + } } @Override |