Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.9.3

Request #8089 Comment review
Submitted: 2006-06-30 13:40 UTC
From: michel dot dhooge at gmail dot com Assigned: wiesemann
Status: Closed Package: Structures_DataGrid (version CVS)
PHP Version: Irrelevant OS:
Roadmaps: (Not assigned)    
Subscription  


 [2006-06-30 13:40 UTC] michel dot dhooge at gmail dot com (Michel D'HOOGE)
Description: ------------ Dear, Here is a patch to improve the comments of DataGrid.php and DataSources. http://www.phpfi.com/126693?lang=patch Cheers Michel

Comments

 [2006-06-30 16:27 UTC] olivierg at php dot net (Olivier Guilyardi)
The DataSource's "field" option is not an "empty array which must be filled-in by the overloading classes". It is meant to be filled by the user, usually when calling bind(). Drivers may (not must) modify it transparently (as DBDO). Why do you duplicate the "fields" option in the DataObject driver ? It is common to all DataSource drivers. Two different documentation entries, at different places, for the same option, is not a good idea. But it looks like you want to say "okay, that's the field option, but with DBDO it behaves in a specific manner". Mark, as you're working on options doc, what do you think about this ? By the way, to learn about our current way of documenting options, please look at HTMLTable.php Apart from that, it looks good to me.
 [2006-07-02 17:53 UTC] wiesemann (Mark Wiesemann)
I agree on Olivier's comment. About the options: We should not duplicate the comments for each driver, that will cause too much headaches for updates. But we could either make one page 'common options for all DataSources' (or Renderers) or we could include such a table below the driver-only options, i.e. first the table with the driver-specific options, then the table with the common options. I prefer the last variant, it makes it easy for people to see all options.
 [2006-07-03 09:29 UTC] michel dot dhooge at gmail dot com
> The DataSource's "field" option is not an "empty > array which must be filled-in by the overloading > classes". OK. My comment would have been more accurate like this: It is empty when the object is created and could be automatically filled-in by some classes (see details of the corresponding class for details). And BTW, the text "(Only used when "generate_columns" is true.)" is neither accurate: DataObject uses the fields options to know which variables must be read in the fetch() method... > Why do you duplicate the "fields" option? As you guessed, I wanted to show the specific behaviour of DBDO. > I prefer the last variant, it makes it easy for > people to see all options. Is is possible to do this automatically with phpDocumentor?
 [2006-07-04 14:36 UTC] olivierg at php dot net (Olivier Guilyardi)
Mark: > About the options: We should not duplicate the comments > for each driver, that will cause too much headaches for > updates. I'm not sure about this actually. It may be necessary to talk about the specific behaviour, side-effects, extra-informations of an option for a specific driver. > i.e. first the table with the driver-specific options, > then the table with the common options. IMO, on the end-user doc, each driver should be associated with a list of options, that transparently contains both common and driver-specific options, all arranged in alphabetical order. A little sentence could mention if this is a common option or not, if necessary. And don't forget that some driver may inherit from both DataSource and DataSource_Array (maybe more in the future). Michel: > And BTW, the text "(Only used when "generate_columns" > is true.)" is neither accurate Right. Fixed in CVS. >> I prefer the last variant, it makes it easy for >> people to see all options. > Is is possible to do this automatically with phpDocumentor? That's precisely what we're talking about. The problem is that in addition to phpDocumentor nice generation features, we also need to provide some sort of "options documentation inheritance", as discussed above. Check tools/manual-gen in the CVS if you're curious about it.
 [2006-07-04 15:53 UTC] wiesemann (Mark Wiesemann)
Okay, valid point, then I would like to suggest the following: - if a global option (e.g. from Renderer.php) has a different meaning in a driver, then the option is with his behaviour also documented in the driver; otherwise the drivers contain only documentation about their own options - make-options.php builds for every driver a table that contains all options he provides (in alphabetial order), i.e. global options, extended options (e.g. from Array), his own options (least position in this orders overwrites earlier defined options) I'll take this into account for my update of parse-options.php.
 [2006-07-04 16:01 UTC] olivierg at php dot net (Olivier Guilyardi)
100% agreed Will your script be able to parse an undefined number of parents, grand-parents, etc ?
 [2006-07-04 16:04 UTC] wiesemann (Mark Wiesemann)
> Will your script be able to parse an undefined number of parents, > grand-parents, etc ? I'll try, let's see. For a first version a simple relation like DataSource.php => Array => CSV is enough, but that might change later.
 [2006-08-17 12:48 UTC] michel dot dhooge at gmail dot com
IMO, the only part that is still of interest if the comment made by Mark on 2006-07-04 15:53 about documentation inheritance.
 [2006-08-17 12:50 UTC] wiesemann (Mark Wiesemann)
The doc inheritance is already implemented, we "just" (in quotes, because it's not a work of a few minutes) need to make full manual pages for the individual drivers.