Banking System for Activity Points

OscarX · Dec 30th 2008

Hello everyone, im currently working on a Banking System for the Activity Points system. I have the event listener and the actual form already.

The only problem im having trouble with is that whenever I try to submit the form, no points are put into the field "banked" in the wcf1_user table.
Anyone have any idea on how I can make a working event listener and submission for such a script.

Im thinking something like this:

PHP

$oldactivityPoints = $activityPoints-$banked;
$newactivityPoints = $activityPoints+$banked;




	/**
	 * Deposit the Rupees
	 */
	public function update($username, $banked = TIME_NOW) {
		$sql = "UPDATE	wcf".WCF_N."_user SET	banked = '".escapeString($banked)."',
 WHERE userID = ".$this->userID;
		WCF::getDB()->sendQuery($sql);
	}

Display More

This is from an old script I had, how can I convert this into the new coding that WBB3 uses?

PHP

$db_connect->query("UPDATE db_users SET usermoney='$newmoney' WHERE userid = '$user_id'");
		$db_connect->query("UPDATE db_users SET userbankmoney='$newbank' WHERE userid = '$user_id'");
		$db_connect->query("UPDATE db_users SET bankinserttime='$time' WHERE userid = '$user_id'");

Thank you for your time!

lookapanda · Dec 31st 2008

Hello OscarX,

did SQL report an error? It might be helpful

greets doenervich

OscarX · Jan 6th 2009

No errors were ever reported. When i send the form the page just reloads and nothing gets saved.

Troublegum · Jan 6th 2009

I think it is unlikely someone can help you unless you provide more detailed information (e.g. the code).

OscarX · Jan 7th 2009

Thats the whole problem, I don't know where to start with the coding part of it. I just drew up a few ideas as you can see in the first post.
Anyone know a code I can use to save a form with a substraction and addition factor in there?

Something like:

PHP

$bankedpoints = $activityPoints-$query;
$withdrawedpoints = $activityPoints+$query;

thank you!

Troublegum · Jan 7th 2009

You should have a look at the file wcf/lib/form/AbstracForm.class.php. This class provides a default implementation for the Form interface.
You can define your own form (you should also take a look at some of our (less complex) forms like EmailChangeForm for example.

PHP

<?php
require_once(WCF_DIR.'lib/form/AbstractForm.class.php');







class MyOwnForm extends AbstractForm {
	public $newmoney = 0;
	public $templateName = 'myownform';





	/**
	 * @see Form::readFormParameters()
	 */
	public function readFormParameters() {
		parent::readFormParameters();

		// use intval for Integer values (or perhaps you would like to use floatval for float values ?)
		if (isset($_POST['newmoney'])) $this->newmoney = intval($_POST['password']);
	}




	/**
	 * @see Form::validate()
	 */
	public function validate() {
		parent::validate();


		// Validate YOUR form input like this:
		if (empty($this->newmoney)) {
			throw new UserInputException('newmoney');
		}

		// or perhaps you would like to accept only positive values for the newmoney variable?
		// like this:
		if ($this->newmoney <= 0) {
			throw new UserInputException('newmoney');
		}

		//
		// ....


	}





	/**
	 * @see Form::save()
	 */
	public function save() {
		parent::save();

		// store data in database
		$sql = "UPDATE	wcf".WCF_N."_user SET usermoney = '".$this->newmoney."'
			WHERE	userID = ".WCF::getUser()->userID;
		WCF::getDB()->sendQuery($sql);
	}



	/**
	 * @see Page::show()
	 */
	public function show() {
		// make sure only registered users can see the form
		if (!WCF::getUser()->userID) {
			require_once(WCF_DIR.'lib/system/exception/PermissionDeniedException.class.php');
			throw new PermissionDeniedException();
		}

		parent::show();
	}

	/**
	 * @see Page::assignVariables()
	 */
	public function assignVariables() {
		parent::assignVariables();

		WCF::getTPL()->assign(array(
			'newmoney' => $this->newmoney,
		));
	}	
}
?>

Display More

You can increment/decrement values in your database like this:

SQL

UPDATE table SET field = field + 35;
UPDATE table SET field = field - 20;

Share