Hi,<br><br>Kindly find the status of the patches which taken up for 5.0.4 release.<br><br><br>
<table style="text-align: left; width: 807px; height: 1411px;" border="1" cellpadding="2" cellspacing="2">
<tbody><tr><td style="vertical-align: top; font-weight: bold;">S.No<br></td><td style="vertical-align: top; font-weight: bold;">Patch<br></td><td style="font-weight: bold;">Contributor<br></td><td style="font-weight: bold;">
File<br></td><td>Message <br></td><td style="vertical-align: top; font-weight: bold;">Status<br></td><td style="vertical-align: top; font-weight: bold;">Comment<br></td></tr><tr><td style="vertical-align: top;">1<br></td>
<td style="vertical-align: top;">new functions and some cleanups
</td><td><a href="mailto:weigelt@metux.de">weigelt@metux.de</a></td><td>01_PearDatabase_newfuncs.diff </td><td>Hi folks,<br><br>
here are some patches (against latest trunk), which introduce <br>
the new functions and so some cleanups. <br><br>
cu</td><td style="vertical-align: top;"> Integrated<br></td><td style="vertical-align: top;"><br></td></tr><tr><td style="vertical-align: top;">2<br></td><td style="vertical-align: top;">new functions and some cleanups
</td><td><a href="mailto:weigelt@metux.de">weigelt@metux.de</a></td><td>3_strip_useless_comments.diff</td><td><br></td><td style="vertical-align: top;"> Integrated</td><td style="vertical-align: top;"><br></td></tr>
<tr><td style="vertical-align: top;">3<br></td><td style="vertical-align: top;">remove noise in invoice module
</td><td><a href="mailto:weigelt@metux.de">weigelt@metux.de</a></td><td>02_nekrad-remove-noise.diff</td><td>Hi folks,<br><br>
this patch removes some noise in the invoice module ...<br><br>
cu</td><td style="vertical-align: top;">Not integrated<br></td><td style="vertical-align: top;">Expected behaviour.<br></td></tr><tr><td style="vertical-align: top;">4<br></td><td style="vertical-align: top;">
PearDatabase.php sanity check
in Move()</td><td><a href="mailto:weigelt@metux.de">weigelt@metux.de</a></td><td>08_nekrad_peardb_sanitycheck-1.diff</td><td>Hi folks,<br><br>
as we sometimes get some errors like:<br><br>
# Fatal error: Call to a member function Move() on a non-object <br>
# in .../include/database/PearDatabase.php on line 443<br><br>
Obviously sometimes the query_result() method is called with an<br>
non-existing result object.<br><br>
This patch adds an check which throws an exception on missing <br>
result object, so we can easily track it down.<br><br><br>
cu</td><td style="vertical-align: top;"> Integrated</td><td style="vertical-align: top;"><br></td></tr><tr><td style="vertical-align: top;">5<br></td><td style="vertical-align: top;">Invoice: fix PDF link
</td><td><a href="mailto:weigelt@metux.de">weigelt@metux.de</a></td><td>invoice-link-pdf.diff<br></td><td>Hi folks,<br><br>
this patch fixes the really strange javascript link for pdf<br>
invoice to an plain url.<br><br>
cu</td><td style="vertical-align: top;">Different fix has been integrated.<br>
</td><td style="vertical-align: top;"><br></td></tr><tr><td style="vertical-align: top;">6<br></td><td style="vertical-align: top;"> some new $adb (PearDatabase) methods
</td><td><a href="mailto:weigelt@metux.de">weigelt@metux.de</a></td><td>09_nekrad_peardb-newfuncs.diff<br></td><td>Hi folks,<br><br>
this patch adds two new functions to the PearDatabase ($adb) <br>
class:<br><br>
* sql_concat(array):<br>
generates an sql expression for field concatenations, based <br>
on selected RDBMS.<br><br>
* run_query_field(string):<br>
executes an given SQL query and returns only the first field<br>
of the first record.<br><br><br>
cu</td><td style="vertical-align: top;"> Integrated</td><td style="vertical-align: top;"><br></td></tr><tr><td style="vertical-align: top;">7<br></td><td style="vertical-align: top;">leads module fix (postgres)
</td><td><a href="mailto:weigelt@metux.de">weigelt@metux.de</a></td><td>31_nekrad-leads-fix.diff<br></td><td>Hi folks,<br><br><br>
the leads module (Tracker.php) suffers from an sql bug: it uses<br>
mysql specific sql statements for concatenating fields. This ends<br>
up in empty result objects and so the already mentioned error<br>
of trying to call Move() on non-object.<br><br>
This patch solves this by using the new PearDatabas::sql_concat()<br>
function (see prev. patch) for creating the statement.<br><br>
Also this patch calls the new PearDatabase::run_query_field()<br>
for the one-field-query.<br><br><br>
cu</td><td style="vertical-align: top;">Integrated with small modification.<br>
</td><td style="vertical-align: top;"><br></td></tr><tr><td style="vertical-align: top;">8<br></td><td style="vertical-align: top;">Link leak in calendar module calendar view</td><td><a href="mailto:jon@eyesopen.com">jon@eyesopen.com
</a></td><td>calendarlink.patch</td><td><br></td><td style="vertical-align: top;">Integrated<br></td><td style="vertical-align: top;"><br></td></tr></tbody>
</table>
<br>Thanks & Regards,<br>Minnie<br><br><div><span class="gmail_quote">On 12/29/07, <b class="gmail_sendername">Enrico Weigelt</b> <<a href="mailto:weigelt@metux.de">weigelt@metux.de</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
* Enrico Weigelt <<a href="mailto:weigelt@metux.de">weigelt@metux.de</a>> schrieb:<br><br>Hi,<br><br>> these patches add an universal rendering API for things like<br>> invoices and an TeX renderer backend. The old invoice pdf code
<br>> is changed to run via the rendering API. Now it's very easy to<br>> add new renderers. I also included my own templates as example.<br>><br>> The API patch (renderapi-1) does not change anything but just
<br>> adding some new files in the lib/render/ subdir and could be<br>> applied to 5.1 and 5.1_jens asap.<br><br>What's the current status on my this patch ?<br><br><br>cu<br>_______________________________________________
<br>Reach hundreds of potential candidates - <a href="http://jobs.vtiger.com">http://jobs.vtiger.com</a><br></blockquote></div><br>