[freeside-devel] MYSQL Patch

Dale Hege fhege at sover.net
Tue Jun 4 07:23:42 PDT 2002


On Tue, 4 Jun 2002, ivan wrote:

> On Mon, Jun 03, 2002 at 06:40:43PM -0400, Dale Hege wrote:
> > Here is most of the MYSQL patch. I havn't installed pre13 yet so I didn't
> > fix the queued bug.
> 
> Well, There's no bug as such in freeside-queued; subqueries are standard
> SQL functionality.  :)
> 

Yeah, yeah, I know. ;)

> Thanks!  This is _almost_ everything that needs to be doing for MySQL
> support.  If you someone could send a patch for freeside-queued, I will
> remove the "MySQL is unsupported" stuff from the documentation and support
> it in the next snapshot.
> 

I'll try and get the queued stuff installed and fixed for MYSQL.

> > But here are at least the searchs.
> > I'm sure I broke something but they seem to work for me. I havn't tested
> > them with PGSQL.
> 
> Applied and checked into CVS, except for the two small sections which
> follow.  Why do you remove the ORDER BY clause in these two places?
> 
> > diff -ur search/cust_pkg.cgi search/cust_pkg.cgi
> > --- search/cust_pkg.cgi Fri May  3 21:11:04 2002
> > +++ search/cust_pkg.cgi Thu May 30 22:09:14 2002
> 
> > -  
> > -  @cust_pkg = qsearch('cust_pkg',{}, '', "$unconf ORDER BY pkgnum $limit" );
> > +  if ( datasrc =~ /mysql/ ) {
> > +         @cust_pkg = qsearch('cust_pkg',{}, '', "$unconf $limit" );
> > +  }else{
> > +         @cust_pkg = qsearch('cust_pkg',{}, '', "$unconf ORDER BY pkgnum $limit" );
> > +  }
> 
> > diff -ur search/svc_acct.cgi search/svc_acct.cgi
> > --- search/svc_acct.cgi Fri Mar 29 12:27:18 2002
> > +++ search/svc_acct.cgi Thu May 30 22:33:19 2002
> 
> > -
> > -  @svc_acct = qsearch('svc_acct', {}, '', "$unlinked $orderby $limit");
> > -
> > +  if ( datasrc =~ /mysql/ ) {
> > +       @svc_acct = qsearch('svc_acct', {}, '', "$unlinked  $limit");  
> > +  }else{
> > +       @svc_acct = qsearch('svc_acct', {}, '', "$unlinked $orderby $limit");
> > +  }
> 

Hmm, I don't remember. I'll fix that. ;)

-Dale




More information about the freeside-devel mailing list