Skip to content

Codecrew / Moya

Go to a project
Merge Request !134 opened by

Placeslot and seatmap stuff

Kaikenmoista mahdollista slotteihin ja paikkakarttaan liittyvää.

HUOM! Ei vielä täysin toimivaa koodia ja paljon läpikäytävää ja testattavaa.

Vilkuili ja hyväksyi: @tkfftk

Edited
Request to merge tuomari:logging into master

Merged

  • The changes were merged into master.
  • The source branch has been removed.
  • @tuomari

    Added 5 new commits:

    • 2ce2e946 - Create private bean variant of BillBean
    • 28cfa136 - Fix verkkomaksut ( paytrail ) api stuff and remove verkkomaksurunner references
    • 2b81bca1 - Parameters in right order.
    • 87465ae3 - Make verkkomaksubuttons a component and fix xhtml part of payments
    • e8b510a7 - Servlet needs to extend HttpServlet if it is wanted to be started at boot

    Added 5 new commits:

    • 2ce2e946 - Create private bean variant of BillBean
    • 28cfa136 - Fix verkkomaksut ( paytrail ) api stuff and remove verkkomaksurunner references
    • 2b81bca1 - Parameters in right order.
    • 87465ae3 - Make verkkomaksubuttons a component and fix xhtml part of payments
    • e8b510a7 - Servlet needs to extend HttpServlet if it is wanted to be started at boot
    Toggle commit list
  • 2a54e04ac87f85d4d091c3be1c8f9cea?s=80&d=identicon
    Tuukka Kivilahti
    @tkfftk started a discussion on the diff
    code/moya-beans/ejbModule/fi/codecrew/moya/beans/PlaceBean.java
    227 238 if (!place.isTaken() || (permbean.hasPermission(MapPermission.MANAGE_OTHERS) && permbean.getCurrentUser().equals(place.getCurrentUser()))) {
    228 239 if (place.isBuyable() || permbean.hasPermission(MapPermission.MANAGE_OTHERS)) {
    229 240  
    241 if (place.getProduct().getProductFlags().contains(ProductFlag.PREPAID_CREDIT)) {
    242 // TODO: We should check there is enough credits...
  • @tkfftk

    mentioned in commit 380c6c51

    mentioned in commit 380c6c51

    Toggle commit list
  • E653e45857f523ba676b9ae7a0bab5dd?s=80&d=identicon
    Tuomas Riihimäki
    @tuomari started a discussion on the diff
    code/moya-beans/ejbModule/fi/codecrew/moya/beans/PlaceBean.java
    227 238 if (!place.isTaken() || (permbean.hasPermission(MapPermission.MANAGE_OTHERS) && permbean.getCurrentUser().equals(place.getCurrentUser()))) {
    228 239 if (place.isBuyable() || permbean.hasPermission(MapPermission.MANAGE_OTHERS)) {
    229 240  
    241 if (place.getProduct().getProductFlags().contains(ProductFlag.PREPAID_CREDIT)) {
    242 // TODO: We should check there is enough credits...
    • @tuomari commented
      Owner

      Tämä on vanhaa koodia. Kredittien tarkastus tehdään tällä hetkellä vähän väärässä paikassa ja jos kaikki on kunnossa kutsutaan vasta tätä funktiota. Kommentti lisätty että muistetaan tehdä oikeasti kredittitarkastelu.
Markdown is supported
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!