[aur-dev] AUR Coding recomendations IMPORTANT

[Loui]

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.