Re: Access Denied on "Add Time Card" in 1.2.15

["Angus Jordan" <..hidden..>]

Thanks for the quick responses.  After sending that last email, I
looked at the branches/1.2 diff and found the fix myself. :)

It seemed like a pretty simple fix, and from what I could tell did
nothing to hurt anything so I just did the "patch" myself.

Keep up the great work guys!

Cheers,
Angus


On Fri, Aug 29, 2008 at 4:11 AM, Jeff Kowalczyk <..hidden..> wrote:
> On Thu, 28 Aug 2008 20:53:33 -0700, Angus Jordan wrote:
>> Will this be resolved before 1.2.16? This is a pretty big issue for us
>> as we use pricematrix and time cards quite heavily...
>>
>> Is there a patch currently available that calls the pricematrix logic
>> correctly? Or is the fix in branches/1.2 OK to use in production?
>
> It was a one-line fix by Seneca and Chris, already committed to SVN r2304:
>
> $ svn diff -c 2304
> Index: LedgerSMB/PriceMatrix.pm
> ===================================================================
> --- LedgerSMB/PriceMatrix.pm    (revision 2303)
> +++ LedgerSMB/PriceMatrix.pm    (revision 2304)
> @@ -122,7 +122,6 @@
>     } elsif ( $form->{vendor_id} ) {
>         $pmh->execute( $ref->{id} );
>     } else {
> -        $form->error('Access Denied');
>         return;
>     }
>
>
>
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
> Build the coolest Linux based applications with Moblin SDK & win great prizes
> Grand prize is a trip for two to an Open Source event anywhere in the world
> http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Ledger-smb-users mailing list
> ..hidden..
> https://lists.sourceforge.net/lists/listinfo/ledger-smb-users
>