From: Dan A. <Dan_Austin@Phoenix.com> - 2006-06-13 21:33:12
|
Don't sweet it. I've been coding like a mad man, as a recent upgrade here finally killed the proprietary conferencing platform we were using. That basically forced management to let me deploy Asterisk and Web-MeetMe. The response internally has been excellent, and goaded me into adding a number of planned features. I've found a couple of issues that involve your code that I could not determine the best way to proceed. 1. _SESSION['userid'] vs _SESSION['username'] You have both, but the code seems to prefer userid, except for displaying the conference owner. The easiest fix would be for LDAP authentication to set both, but I'm not clear on why both are needed. 2. The direct MySQL queries.=20 I really like the functionality you added using them, and=20 I've done the same in a couple of new features, but this breaks the database agnostic approach, and likely ends support for postgres. While I don't use postgres, I think we should try to maintain the flexibility. Oh, and I just added a redirect on my home server to point people looking for the package to SF and saw the stats jump quite a bit. Dan -----Original Message----- From: Mike Clark [mailto:mik...@wp...]=20 Sent: Tuesday, June 13, 2006 2:18 PM To: Dan Austin; Web...@li... Subject: Re: [Web-meetme-devel] testing Dan Austin wrote: > SF claims that the mail list issues have been resolved..... >=20 >=20 > _______________________________________________ > Web-meetme-devel mailing list > Web...@li... > https://lists.sourceforge.net/lists/listinfo/web-meetme-devel >=20 Dan: Just shoot me. I apologize for not getting back sooner. Things have been crazy lately. I did get your prior emails. I will review tonight at the=20 hotel and reply. I am just completing a major IVR project, so I hope my=20 schedule will be opening up more over the next few weeks. Mike |