Skip to content
  • Projects
  • Groups
  • Snippets
  • Help

Codecrew / Moya

  • This project
    • Loading...
  • Sign in
Go to a project
  • Project
  • Repository
  • Issues 30
  • Merge Requests 2
  • Wiki
  • Snippets
  • Settings
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Commits
  • Issue Boards
Merged
Merge Request !54 opened Apr 12, 2014 by Tuomas Riihimäki@tuomari

Credit transfer (and some other stuff)

Tuli taas vähän turhan iso, mutta

  • Käyttäjien ID export tapahtumasta käyttäjäkorin kautta
    • Puolipisteellä erotettu lista User.Id:itä
  • Käyttäjien import edellisestä exportista tapahtumaan.
  • Tehty loppuun credittisiirto tapahtumasta toiseen. ( Käyttäjäkorin kautta. )
  • Fiksailtu ja uudelleenkirjoitettu osittain käyttäjien listaus (paginointi ja sorttaus primefacesin päälle)
  • Pieniä viilauksia tilitapahtuman esittämiseen. ( descriptionit )
  • Ei luoda ihan aina automaattisesti uutta käyttäjää tapahtumaan.
Edited Apr 15, 2014
Request to merge tuomari:credittransfer into master

Merged

  • The changes were merged into master.
  • The source branch has been removed.
  • Discussion 3
  • Commits 5
  • Changes 49
  • {{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
  • 2a54e04ac87f85d4d091c3be1c8f9cea?s=80&d=identicon
    Tuukka Kivilahti
    @tkfftk started a discussion on the diff Apr 15, 2014
    code/MoyaBeans/ejbModule/fi/codecrew/moya/beans/UserBean.java
    785 804 public User getUser(Integer id) {
    786 805  
    787 806 User ret = userFacade.find(id);
    788 if (!permbean.getCurrentUser().getUser().equals(ret) && permbean.hasPermission(UserPermission.VIEW_ALL)) {
    807 if (!permbean.getCurrentUser().getUser().equals(ret) && !permbean.hasPermission(UserPermission.VIEW_ALL)) {
    789 808 throw new EJBAccessException("Tried to fetch user with id " + id + " from database without sufficient permissions");
    790 809 }
    791 810  
    792 811 return ret;
    793 812 }
    813
    814 @Override
    815 @RolesAllowed(EventPermission.S_MANAGE_EVENT)
    816 public BigDecimal transferAccountSaldoFromPreviousEvent(List<User> users, LanEvent source)
    • Tuukka Kivilahti @tkfftk commented Apr 15, 2014
      Owner

      Tää vois logittaa kans notta kuka siirtää, ja esmes. audit-logiin ois kiva saada jotain

  • 2a54e04ac87f85d4d091c3be1c8f9cea?s=80&d=identicon
    Tuukka Kivilahti
    @tkfftk started a discussion on the diff Apr 15, 2014
    code/MoyaBeans/ejbModule/fi/codecrew/moya/facade/UserFacade.java
    142 143 return getEm().createQuery(cq).getResultList();
    143 144 }
    144 145  
    146 /**
    147 * For example in sorting.
    148 *
    149 * @param fieldname
    150 * @return By default return "id"
    151 */
    152 public static SingularAttribute<? super User, ?> getUserField(String fieldname) {
    153 SingularAttribute<? super User, ?> sort = User_.id;
    154
    155 if (fieldname != null)
    156 {
    157 switch (fieldname) {
    158 case "created":
    • Tuukka Kivilahti @tkfftk commented Apr 15, 2014
      Owner

      Oikeesti? tätä ei voi automatisoida?

  • 2a54e04ac87f85d4d091c3be1c8f9cea?s=80&d=identicon
    Tuukka Kivilahti
    @tkfftk started a discussion on the diff Apr 15, 2014
    code/MoyaWeb/src/fi/codecrew/moya/HostnameFilter.java
    161 161  
    162 162 protected void parseHostname(HttpServletRequest httpRequest)
    163 163 {
    164 logger.info("Path info {}", httpRequest.getPathInfo());
    165 logger.info("querystring {}", httpRequest.getQueryString());
    166 logger.info("ctxpath {}", httpRequest.getContextPath());
    167 logger.info("pathTranslated {}", httpRequest.getPathTranslated());
    168 logger.info("requestUri {}", httpRequest.getRequestURI());
    169 logger.info("URL {}", httpRequest.getRequestURL().toString());
    170 logger.info("servletpath {}", httpRequest.getServletPath());
    171 logger.info("servletCtx {}", httpRequest.getServletContext());
    164 // logger.info("Path info {}", httpRequest.getPathInfo());
    • Tuukka Kivilahti @tkfftk commented Apr 15, 2014
      Owner

      Oikein \o/

  • Write
  • Preview
Markdown is supported
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
  • Please register or sign in to post a comment
Assignee
No assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
0
Labels
None
Assign labels
  • View labels
2
2 participants
Reference: codecrew/moya!54