On Fri, Jun 20, 2008 at 07:54:24AM +0200, =?ISO-8859-1?Q?Angel_Vel=E1squez_ wrote:
Hi, I am writing this e-mail after reviewing the AUR code and some patches that people have been sending.
As we know, AUR (like many applications) is still in development, and many of the things that are actually working weren't coded by any of us, and I can understand this.
I think AUR is more in maintenance mode more than development mode. Developers of AUR have been envisioning a better system for awhile but just haven't quite gotten to it. I really just want to get AUR to a cool state where I can devote time to developing an even awesomer system than what currently exists. Hey maybe you could be the next maintainer if you send enough patches. *wink wink*
The purpose of my e-mail is to give some recommendations that should be adopted by everyone, I don't want to be arrogant or force you to do something without knowing why, so I will explain the reasons behind my recommendations and then you will probably agree with me. [...] 4.- // the last example of a block of HTML print "<table>\n"; print "<tr>\n"; print "<td>".$title."</td>\n"; print "</tr>\n"; print "</table>\n"; // Come on!!!!, the following line does the same.
echo "<table>\n<tr>\n<td>$title</td>\n</tr>\n</table>"; // Yes I know that placing plenty of \n's would render the code unreadable, but you can do it in one line with a simple echo.
A much better way to clean this up would be: <table> <tr> <td><?php echo $title; ?></td> </tr> </table> And that's what we're trying to do. Just take as much HTML out of PHP as possible. Thanks for your suggestions. Rock on.