http://pear.php.net/bugs/12261 [Closed] xsl transformation chooses wrong value for length en-us pear-webmaster@lists.php.net pear-webmaster@lists.php.net hourly 1 2000-01-01T12:00+00:00 http://pear.php.net/bugs/12261 MDB2_Schema Documentation Problem Reported by ggramlich 2007-10-16T19:33:59-00:00 PHP: Irrelevant OS: Package Version: 0.8.1 Description: ------------ The xsl transformation shows the value of the first <length> elment for each <field> with a <length> in a <table>. In MDB2_Schema/docs/schema2html.xsl change line 147 from <xsl:template name="showlength">(<xsl:value-of select="//length"/>) to <xsl:template name="showlength">(<xsl:value-of select="current()"/>) (or to <xsl:template name="showlength">(<xsl:value-of select="../length"/>) )]]> MDB2_Schema Documentation Problem Reported by ggramlich 2007-10-16T19:33:59-00:00 PHP: Irrelevant OS: Package Version: 0.8.1 Description: ------------ The xsl transformation shows the value of the first <length> elment for each <field> with a <length> in a <table>. In MDB2_Schema/docs/schema2html.xsl change line 147 from <xsl:template name="showlength">(<xsl:value-of select="//length"/>) to <xsl:template name="showlength">(<xsl:value-of select="current()"/>) (or to <xsl:template name="showlength">(<xsl:value-of select="../length"/>) )]]> 2007-10-16T19:33:59-00:00 http://pear.php.net/bugs/12261#1192656051 This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.]]> This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.]]> 2007-10-17T21:20:51-00:00 http://pear.php.net/bugs/12261#1192655795 Yes <foreign> is already adopted sice 0.8.0. xml_schema_documentation.html is the most reliable source for MDB2_Schema features. Pretty much everything shown in there should be already implemented. So you may want to report a new documentation problem for the DB package. Regarding your suggestions about the new XML elements please open a new feature request. regards, iGor.]]> Yes <foreign> is already adopted sice 0.8.0. xml_schema_documentation.html is the most reliable source for MDB2_Schema features. Pretty much everything shown in there should be already implemented. So you may want to report a new documentation problem for the DB package. Regarding your suggestions about the new XML elements please open a new feature request. regards, iGor.]]> 2007-10-17T21:16:35-00:00 http://pear.php.net/bugs/12261#1192615458 I am not sure, if what is written here: http://pear.php.net/manual/en/package.database.db-table.intro-database.php Quote: The XML schema used by these methods is an extension of the current MDB2_Schema DTD, extended so as to allow specification of foreign key references. This extension for foreign keys has been agreed upon for adoption in a future release of MDB2_Schema. End quote. is already adopted, since there is a <foreign> section in http://cvs.php.net/viewvc.cgi/pear/MDB2_Schema/docs/xml_schema_documentation.html?view=co So maybe the docs should be updated for DB_Table_Database. Suggestions: <comments> and maybe <description> elements for <index> as well as for <foreign>. Right now I don't see other desirable changes.]]> I am not sure, if what is written here: http://pear.php.net/manual/en/package.database.db-table.intro-database.php Quote: The XML schema used by these methods is an extension of the current MDB2_Schema DTD, extended so as to allow specification of foreign key references. This extension for foreign keys has been agreed upon for adoption in a future release of MDB2_Schema. End quote. is already adopted, since there is a <foreign> section in http://cvs.php.net/viewvc.cgi/pear/MDB2_Schema/docs/xml_schema_documentation.html?view=co So maybe the docs should be updated for DB_Table_Database. Suggestions: <comments> and maybe <description> elements for <index> as well as for <foreign>. Right now I don't see other desirable changes.]]> 2007-10-17T10:04:18-00:00 http://pear.php.net/bugs/12261#1192613780 Currently our XML schema doesn't allow neither a comment or a description for indices. more here: http://cvs.php.net/viewvc.cgi/pear/MDB2_Schema/docs/xml_schema_documentation.html?view=markup description.schema.xml is not touched for a long time. please let me know if you want to suggest another change before I commit the fix. regards, iGor.]]> Currently our XML schema doesn't allow neither a comment or a description for indices. more here: http://cvs.php.net/viewvc.cgi/pear/MDB2_Schema/docs/xml_schema_documentation.html?view=markup description.schema.xml is not touched for a long time. please let me know if you want to suggest another change before I commit the fix. regards, iGor.]]> 2007-10-17T09:36:20-00:00 http://pear.php.net/bugs/12261#1192604709 Thanks for the quick fix. I re-opened the bug, instead of creating a new one: The same problem applies to MDB2_Schema/docs/MDB.xsl line 90. Change "//length" to "current()". One more point about documentation: neither the MDB.dtd nor the MDB.xsd allow <comments> for <index>, but the description.schema.xml uses <comments> and schema2html.xsl renders it. I think, the dtd and xsd should be changed accordingly. Maybe even a <description> should be allowed for <index>.]]> Thanks for the quick fix. I re-opened the bug, instead of creating a new one: The same problem applies to MDB2_Schema/docs/MDB.xsl line 90. Change "//length" to "current()". One more point about documentation: neither the MDB.dtd nor the MDB.xsd allow <comments> for <index>, but the description.schema.xml uses <comments> and schema2html.xsl renders it. I think, the dtd and xsd should be changed accordingly. Maybe even a <description> should be allowed for <index>.]]> 2007-10-17T07:05:09-00:00 http://pear.php.net/bugs/12261#1192566574 This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.]]> This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.]]> 2007-10-16T20:29:34-00:00