Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Fix for #191 #192

Closed
wants to merge 2 commits into from

2 participants

@tjlytle

Temporary fix for #191.

tjlytle added some commits
@Gipetto
Owner

Fixed. Closing.

@Gipetto Gipetto closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 28, 2012
  1. @tjlytle

    Temporary fix for #191

    tjlytle authored
Commits on Jan 31, 2012
  1. @tjlytle

    Better fix for #191

    tjlytle authored
    Checking if the id is an integer.
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  OpenVBX/libraries/MY_Model.php
View
2  OpenVBX/libraries/MY_Model.php
@@ -388,7 +388,7 @@ function delete()
function save($force_update = false)
{
- if(intval($this->id) > 0)
+ if(is_numeric($this->id) AND intval($this->id) > 0)
{
$this->update($this->id, $this->values);
return true;
Something went wrong with that request. Please try again.