Oppdaget feil ved salg av nytt medlemskort til livsvarige medlemmer.
Bruk hasExpired() i stedet for å sjekke på dato for å sjekke medlemsstatus ved medlemskapssalg. hasExpired() har støtte for livsvarig medlemskap.
This commit is contained in:
1 parent
5f36bf87e0
commit
58608a4ea5
1 file changed
+7
-8
+7
-8
@@ -575,11 +575,11 @@ class User {
|
||||
}
|
||||
|
||||
public function renewMembership($cardno) {
|
||||
$expires = $this->getExpiryDate();
|
||||
if ($expires <= $this->expires) {
|
||||
if (!$this->hasExpired()) {
|
||||
notify("Du har allerede gyldig medlemskap for dette året.");
|
||||
return false;
|
||||
}
|
||||
$expires = $this->getExpiryDate();
|
||||
$this->conn->autoCommit(false);
|
||||
if ($this->_registerUsedCardno($cardno) == true) {
|
||||
$sql = "UPDATE din_user SET " . " expires = '$expires' " . "WHERE " . " id = $this->id";
|
||||
@@ -602,11 +602,11 @@ class User {
|
||||
}
|
||||
|
||||
public function renewMembershipPayex() {
|
||||
$expires = $this->getExpiryDate();
|
||||
if ($expires <= $this->expires) {
|
||||
if (!$this->hasExpired()) {
|
||||
notify("Du har allerede gyldig medlemskap for dette året.");
|
||||
return false;
|
||||
}
|
||||
$expires = $this->getExpiryDate();
|
||||
$this->conn->autoCommit(false);
|
||||
$sql = "UPDATE din_user SET " . " expires = '$expires' " . "WHERE " . " id = $this->id";
|
||||
$result = $this->conn->query($sql);
|
||||
@@ -626,11 +626,11 @@ class User {
|
||||
}
|
||||
|
||||
public function renewMembershipEurobate() {
|
||||
$expires = $this->getExpiryDate();
|
||||
if ($expires <= $this->expires) {
|
||||
if (!$this->hasExpired()) {
|
||||
// har allerede gyldig medlemskap
|
||||
return false;
|
||||
}
|
||||
$expires = $this->getExpiryDate();
|
||||
$this->conn->autoCommit(false);
|
||||
$sql = "UPDATE din_user SET " . " expires = '$expires' " . "WHERE " . " id = $this->id";
|
||||
$result = $this->conn->query($sql);
|
||||
@@ -685,8 +685,7 @@ class User {
|
||||
|
||||
/* Order new membercard if old one is lost */
|
||||
public function renewMembercardPayex() {
|
||||
$expires = $this->getExpiryDate();
|
||||
if ($expires > $this->expires) {
|
||||
if ($this->hasExpired()) {
|
||||
notify("Du har ikke gyldig medlemskap for dette året.");
|
||||
return false;
|
||||
}
|
||||
|
||||
Reference in new issue
Block a user