bugphpGroupWare - Bugs: bug #4090, problem with createobject if...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #4090: problem with createobject if classes are called 'appname.ui'

Submitted by:  Bettina Gille <ceb>
Submitted on:  Tue 24 Jun 2003 09:43:22 PM UTC  
 
Item Group: SVN trunk (dev code)Category: API - phpGroupware API
Severity: 3 - NormalPriority: 5 - Normal
Status: FixedPrivacy: Public
Assigned to: NoneOpen/Closed: Closed
Component Version: SVNOperating System: None
Reproducibility: NonePlanned Release: None
Fixed Release: 

Thu 17 Jul 2003 02:22:52 AM UTC, comment #5:

Closing this one. Michael has fixed his app. I add something to the wiki too.

Dave Hall <skwashd>
Project Administrator
Sun 29 Jun 2003 07:32:22 AM UTC, comment #4:

i think it is handy for all apps to use uiapp* for at least their ui classes to reduce namespace conflicts - until PHP5 takes over :)

Anonymous
Sat 28 Jun 2003 11:15:43 PM UTC, comment #3:

it depends. classes bonews and boadmin, and sonews and soadmin duplicated a lot of fonctionality, but in a very inconsistent way, so it made sense to standardize and merge them. I merged classes uinews and uiadmin for symetry's sake, but I could have let them separate.

Michael Totschnig <totschnig>
Sat 28 Jun 2003 03:48:15 PM UTC, comment #2:

You also don't want to necessarily pile all of your HTML code into one class. Splitting the different screens out into their own classes would keep the files smaller and more maintainable. It also would help to force picking real names instead of generic ones ;-)

Michael L. Dean <mdean>
Sat 28 Jun 2003 04:31:41 AM UTC, comment #1:

I will rename news_admin.ui back to news_admin.uinews. (I had named it ui when I merged the two classes uinews and uiadmin). sorry for the trouble.

Michael Totschnig <totschnig>
Tue 24 Jun 2003 09:43:22 PM UTC, original submission:

example: home-screen -> hook home;
several apps called their ui class just 'ui'. i.e. news_admin and stocks. in their hook_homes they both create an object 'appname.ui'. it fails to create the second object, because there exists already an object 'ui'. we have to rename our app_classes to i.e. uiappname (uistocks,uinews) or change the createobject function...

Bettina Gille <ceb>
Project Administrator

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

Do you think this task is very important?
If so, you can click here to add your encouragement to it.
This task has 0 encouragements so far.

Only logged-in users can vote.

 

Please enter the title of George Orwell's famous dystopian book (it's a date):

 

 

Follow 2 latest changes.

Date Changed By Updated Field Previous Value => Replaced By
Thu 17 Jul 2003 02:22:52 AM UTCskwashdStatusNone=>(Error - Not Found)
  Open/ClosedOpen=>(Error - Not Found)

Back to the top


Powered by Savane 3.1-cleanup1