Arnaud Limbourg | 1 May 2005 10:21

Re: Structures_Datagrid: new DataSource class

Hi,

I think the best way to go is to post a feature request on pear bugs.
There is a link on the package homepage.

This way it doesn't ge lost.

Arnaud.

Mark Wiesemann wrote:
> Hi,
> 
> I'd like to propose a new DataSource class which can handle DB_Table
> objects.
> 
> Here is the code: http://www.markwiesemann.de/php/DB_Table.phps
> 
> The code bases on the DB_Result DataSource class. As DB_Table can handle
> sorting in the select* functions the sort function of my class is not
> needed. At the moment a PEAR_Error object is returned if one calls this
> function. I don't know enough of the internals of Datagrid - maybe the
> function is not needed in every DataSource driver class and can be
> deleted.
> 
> Usage of the new driver:
> 
> (1) Assuming one has a DB_Table object called $c:
> 
> require_once 'Structures/DataGrid/DataSource.php';
> $dg =& new Structures_DataGrid();
(Continue reading)

Mark Wiesemann | 1 May 2005 12:22
Picon

Re: Structures_Datagrid: new DataSource class

> I think the best way to go is to post a feature request on pear bugs.
> There is a link on the package homepage.

Done: http://pear.php.net/bugs/bug.php?id=4254

I have posted the proposal here because the manual [1] says:

"If you do create your own driver and find that it may be useful to
others, please post it to the mailing list so that it could be bundled
into the package."

But using the bugtracker makes also very much sense.

Mark

[1]
http://pear.php.net/manual/en/package.structures.structures_datagrid.structures_datagrid.datasource.create.php

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Arnaud Limbourg | 1 May 2005 14:24

Re: Structures_Datagrid: new DataSource class

> I have posted the proposal here because the manual [1] says:
> 
> "If you do create your own driver and find that it may be useful to
> others, please post it to the mailing list so that it could be bundled
> into the package."
> 
> But using the bugtracker makes also very much sense.

I think doing both is the best way to go. It gets noticed by being sent to the list and not forgotten by being in
the tracker :)

Arnaud.

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

bertrand Gugger | 1 May 2005 14:24
Favicon

Archive_Ar

Pablo,
Great proposal !
http://pear.php.net/pepr/pepr-proposal-show.php?id=243
I had hoped it coupled with File_Archive...
I was wrong as your target is the Debian world.

Just, (after a quick glance) never echo ! But give back the string !

Mandibulas te comera !

Buenas !
-- 
bertrand Gugger
http://toggg.com

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Pablo Fischer | 1 May 2005 19:26
Picon

[PEPr] Proposal for File Formats::Archive_Ar


Pablo Fischer (http://pear.php.net/user/pfischer) proposes File Formats::Archive_Ar.

You can find more detailed information here:
 http://pear.php.net/pepr/pepr-proposal-show.php?id=243

-- 
Sent by PEPr, the automatic proposal system at http://pear.php.net

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Justin Patrin | 1 May 2005 19:45
Gravatar

[PEPr] Comment on File Formats::Archive_Ar


Justin Patrin (http://pear.php.net/user/justinpatrin) has commented on the proposal for File Formats::Archive_Ar.

Comment:

This format is also used for the ipkg format (which is basically the same
as a Debian Archive) which is used for OpenZaurus, Familiar, and
OpenSimpad.

I don't think that printFiles method should really be in there. If a user
want to print a table listing of the files it's just as easy to use
Structures_DataGrid and the file array.

The keys of the file array are a bit strange. 'sz' for size seems a bit
simple. Why not just ise 'size'? Also, 'modt' instead of 'mtime'...why not
just use the standard 'mtime'?

Having a local _isError function is useless as you're already requiring
PEAR.php. Just use PEAR::isError().

I would personally rather you used ' instead of " for all strings that
don't need ". i.e. all strings without \n or variables in them.

Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=243

-- 
Sent by PEPr, the automatic proposal system at http://pear.php.net

--

-- 
(Continue reading)

Matthew Fonda | 1 May 2005 21:09
Picon

[PEPr] Comment on File Formats::Archive_Ar


Matthew Fonda (http://pear.php.net/user/mfonda) has commented on the proposal for File Formats::Archive_Ar.

Comment:

I have not looked closely, but if there is any chance that this could some
how be added into File_Archive I think that would be a good idea.

Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=243

-- 
Sent by PEPr, the automatic proposal system at http://pear.php.net

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Pablo Fischer | 1 May 2005 23:11
Picon

[PEPr] Comment on File Formats::Archive_Ar


Pablo Fischer (http://pear.php.net/user/pfischer) has commented on the proposal for File Formats::Archive_Ar.

Comment:

I've uploaded a pre-package, devels can grab it from:

http://pablo.com.mx/pear/Archive_Ar.tar.gz

It fixes some bugs, adds some features and my first Pear package ;-). It
also includes tests (in phpt), samples and doc (in docbook).

 * Structures_DataGrid's are better than my crap printFiles, so I removed
it.

 * Renamed the keys of the array. I was using the same keys that are
documented on on the API (of Ar File Format). But I'm agree in using other
keys.

 * _isError is deprecated, PEAR::isError will be used.

 * Ar files can be created, this wasn't possible (an Ar file was
required).

 * Fixes minor bugs.

Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=243

--

-- 
(Continue reading)

Lukas Smith | 1 May 2005 23:16

[PEPr] Comment on File Formats::Archive_Ar


Lukas Smith (http://pear.php.net/user/lsmith) has commented on the proposal for File Formats::Archive_Ar.

Comment:

However there is no value in extending from PEAR unless you use error
codes. The main advantage of extending from PEAR is 1) destructor
emulation (which is not used here) and 2) expectError(). However
expectError() only works with error codes.

For error codes you would best define a set of constants that you pass to
raiseError().

However if you define error codes you should probably also define your own
error class so that people can figure out from an error object what error
code map to use.

Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=243

-- 
Sent by PEPr, the automatic proposal system at http://pear.php.net

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Lukas Smith | 1 May 2005 23:19

[PEPr] Comment on File Formats::Archive_Ar


Lukas Smith (http://pear.php.net/user/lsmith) has commented on the proposal for File Formats::Archive_Ar.

Comment:

Check this thread for reference:

http://marc.theaimsgroup.com/?l=pear-dev&m=110754527511354&w=2

Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=243

-- 
Sent by PEPr, the automatic proposal system at http://pear.php.net

--

-- 
PEAR Development Mailing List (http://pear.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php


Gmane