improve performance by avoiding node_load

Interesting idea, I've done the same for debugging and refactoring, but for performance is a pretty cool idea.

To be even more seamless, should probably implement the other common magic methods

__isset()
__unset()

and extend stdClass.

Not sure if there is a way to handle things like print_r, cast to array, serialize, etc, but that might be OK.

+++ b/spaces_og/plugins/space_og.incundefined
@@ -191,3 +191,117 @@ class space_og extends space_type_purl {
+  public function __isset($property) {

Could probably be

  return ($this->__get($property) === NULL);

+++ b/spaces_og/plugins/space_og.incundefined
@@ -191,3 +191,106 @@ class space_og extends space_type_purl {
+    elseif (!$this->loaded) {
+      switch ($requested_property) {
+        case 'og_private':
+          $sql = 'SELECT og_private FROM {og} WHERE nid = %d';
+          $properties = db_fetch_array(db_query($sql, $this->node->nid));
+          foreach ($properties as $property => $value) {
+            $this->node->$property = $value;
+          }

If it isn't loaded, and the first property it asks for is $node->og_private, this would probably fatal since $this->node would still be null at this point. I think you should at least pre-define $this->node to be a new stdClass() to avoid this.

Disregard ^.... Didn't see you initialized it in the constructor.