| Index: trunk/extensions/MarkAsHelpful/includes/MarkAsHelpfulItem.php |
| — | — | @@ -22,7 +22,7 @@ |
| 23 | 23 | ); |
| 24 | 24 | |
| 25 | 25 | protected $user; |
| 26 | | - protected $loadFromDatabase = false; |
| | 26 | + protected $loadedFromDatabase = false; |
| 27 | 27 | |
| 28 | 28 | /** |
| 29 | 29 | * Constructor |
| — | — | @@ -47,7 +47,7 @@ |
| 48 | 48 | |
| 49 | 49 | public function getUser() { |
| 50 | 50 | if ( !$this->user ) { |
| 51 | | - if ( $this->loadFromDatabase ) { |
| | 51 | + if ( $this->loadedFromDatabase ) { |
| 52 | 52 | if ( $this->getProperty( 'mah_user_id' ) ) { |
| 53 | 53 | $this->user = User::newFromId( $this->getProperty( 'mah_user_id' ) ); |
| 54 | 54 | } elseif ( $this->getProperty( 'mah_user_ip' ) ) { |
| — | — | @@ -130,9 +130,7 @@ |
| 131 | 131 | $this->setProperty( $key, $res->$key ); |
| 132 | 132 | } |
| 133 | 133 | |
| 134 | | - // @todo FIXME: $Item is not defined (and it should be called $item |
| 135 | | - // as per our coding conventions) |
| 136 | | - $Item->setLoadFromDatabase = true; |
| | 134 | + $this->loadedFromDatabase = true; |
| 137 | 135 | |
| 138 | 136 | return true; |
| 139 | 137 | } else { |