problem adding a field to an object

lkasdorf

unread,

Jul 7, 2014, 4:53:21 PM7/7/14

to php-object...@googlegroups.com

I have a pretty complex set of objects with child-parent and sibling relationships, etc. Just recently, I tried adding a couple fields (like I have been doing for many years now in POG), and the resulting code is broken! Example:

/**

* Associates the account object to this one

* @return boolean

*/

function GetAccount()

{

$account = new ();

return $account->Get($this->accountId);

}

Notice the

new ();

- it should say

new account();

I know Joel is not supporting POG any more, but I have become dependent on it- I have used it dozens and dozens of time over the last 6 years or so.

Mario

unread,

Jul 7, 2014, 5:17:14 PM7/7/14

to php-object-generator

Hi lkasdorf,

Thanks for reporting the error.

Please accept my apologies for this problem.

I guess the error comes from line 800 of

https://github.com/joelwan/php-object-generator/blob/master/object_factory/class.objectphp5.1pdomysql.php

where

$this->string .= "\n\t\t\$".strtolower($parent)." = new ".$class."();";

should be:

$this->string .= "\n\t\t\$".strtolower($parent)." = new ".$parent."();";

as it is on line 786 of file

https://github.com/joelwan/php-object-generator/blob/master/object_factory/class.objectphp5pogmysql.php

now it's too late (11PM) for me to check if this is the best way to solve this error.

I'll check it tomorrow in the evening.

Sorry again for the inconvenience, but we'll be working on it soon.

lkasdorf

unread,

Jul 7, 2014, 5:22:48 PM7/7/14

to php-object...@googlegroups.com

Here is the link:

Thanks!

Lynn Kasdorf

Mario

unread,

Jul 8, 2014, 5:44:39 AM7/8/14

to php-object-generator

hello Lynn,

can you please check you have fulfil the four textboxes for the classes like in this example? [a,b,c,d]

both domains beta.phpobjectgenerator.com and www.phpobjectgenerator.com seem to generate good code for your link.

Please let me know if that was what caused the error.

Have a nice day.

lkasdorf

unread,

Jul 8, 2014, 10:58:46 AM7/8/14

to php-object...@googlegroups.com

yes- that was it- I was not filling in those fields for the related objects. When I put them in, the code produced was correct.

Note that I don't recall having to enter these table names before- I though by default they would use the same name as the field.

Thanks for looking into this so quickly!

Mario

unread,

Jul 8, 2014, 1:48:18 PM7/8/14

to php-object...@googlegroups.com

it's good to know everything went good.

If I have to be honest I don't remember when that feature was included.

The class may not have the same name as the attribute, so it's need to know which one is. This is the reason why you are prompted for it.

Maybe the link should include this information. I'll take into consideration if I change this code.

Have a nice day.

--
You received this message because you are subscribed to the Google Groups "Php Object Generator" group.
To unsubscribe from this group and stop receiving emails from it, send an email to php-object-gener...@googlegroups.com.
To post to this group, send email to php-object...@googlegroups.com.
Visit this group at http://groups.google.com/group/php-object-generator.
For more options, visit https://groups.google.com/d/optout.

Reply all

Reply to author

Forward