[aur-dev] [PATCH 2/3] pkgfuncs.inc.php: allow all DB funcs to take handle arg

[Dan McGee]

On Thu, Aug 11, 2011 at 8:24 AM, Lukas Fleischer
<archlinux at cryptocrack.de> wrote:
> On Wed, Aug 10, 2011 at 06:20:06PM -0500, Dan McGee wrote:
>> Allows handle reuse if one is available.
>>
>> Signed-off-by: Dan McGee <dan at archlinux.org>
>> ---
>>  web/lib/pkgfuncs.inc.php |  164 ++++++++++++++++++++++++++++------------------
>>  1 files changed, 101 insertions(+), 63 deletions(-)
>>
>
> This one looks good to me, except for two lines you probably removed
> unintentionally (see below). This also needs to be rebased on the
> package merging patches in my working tree. I fixed the missing lines
> issue and pushed a rebased version of this patch to my working branch.
I see and updated lukas/wip but not lukas/working, is that what you meant?

>> @@ -276,15 +300,15 @@ function pkgname_is_blacklisted($name) {
>>
>>  # display package details
>>  #
>> -function package_details($id=0, $SID="") {
>> -     $atype = account_from_sid($SID);
>> -     $uid = uid_from_sid($SID);
>
> These two lines shouldn't be removed.

I'll make it more clear and increase my patch count, but they most
definitely should and it wasn't unintentional at all. It also removes
two completely useless queries for every single package load (actually
4 total if I interpreted the code right when writing this patch, but
that is a different story). Find where these variables are used and
I'll buy you a beer.

Anyway, split into two patches, based on master, on my working branch.
If you want rebased patches, then push the work you want it based on
to master please- it isn't fun to aim at moving targets, which is what
I consider any other working branch.

-Dan