Bug fixes

This commit is contained in:
Hillel Coren 2015-11-16 14:30:44 +02:00
parent d68b78f926
commit e93a7aa915
5 changed files with 20 additions and 17 deletions

View File

@ -842,7 +842,7 @@ class AccountController extends BaseController
{ {
$rules = array( $rules = array(
'name' => 'required', 'name' => 'required',
'logo' => 'sometimes|max:512|mimes:jpeg,gif,png', 'logo' => 'sometimes|max:200|mimes:jpeg,gif,png',
); );
$validator = Validator::make(Input::all(), $rules); $validator = Validator::make(Input::all(), $rules);

View File

@ -49,7 +49,7 @@ class PublicClientController extends BaseController
} }
$invoice = $invitation->invoice; $invoice = $invitation->invoice;
$query = $this->activityRepo->findByClientPublicId($invoice->client->public_id); $query = $this->activityRepo->findByClientId($invoice->client_id);
$query->where('activities.adjustment', '!=', 0); $query->where('activities.adjustment', '!=', 0);
return Datatable::query($query) return Datatable::query($query)

View File

@ -61,7 +61,7 @@ class ActivityRepository
return $activity; return $activity;
} }
public function findByClientPublicId($clientPublicId) public function findByClientId($clientId)
{ {
return DB::table('activities') return DB::table('activities')
->join('users', 'users.id', '=', 'activities.user_id') ->join('users', 'users.id', '=', 'activities.user_id')
@ -70,8 +70,7 @@ class ActivityRepository
->leftJoin('invoices', 'invoices.id', '=', 'activities.invoice_id') ->leftJoin('invoices', 'invoices.id', '=', 'activities.invoice_id')
->leftJoin('payments', 'payments.id', '=', 'activities.payment_id') ->leftJoin('payments', 'payments.id', '=', 'activities.payment_id')
->leftJoin('credits', 'credits.id', '=', 'activities.credit_id') ->leftJoin('credits', 'credits.id', '=', 'activities.credit_id')
->where('clients.account_id', '=', Auth::user()->account_id) ->where('clients.id', '=', $clientId)
->where('clients.public_id', '=', $clientPublicId)
->where('contacts.is_primary', '=', 1) ->where('contacts.is_primary', '=', 1)
->whereNull('contacts.deleted_at') ->whereNull('contacts.deleted_at')
->select( ->select(

View File

@ -18,7 +18,9 @@ class ActivityService extends BaseService
public function getDatatable($clientPublicId = null) public function getDatatable($clientPublicId = null)
{ {
$query = $this->activityRepo->findByClientPublicId($clientPublicId); $clientId = Client::getPrivateId($clientPublicId);
$query = $this->activityRepo->findByClientId($clientId);
return $this->createDatatable(ENTITY_ACTIVITY, $query); return $this->createDatatable(ENTITY_ACTIVITY, $query);
} }

View File

@ -155,17 +155,19 @@
</div> </div>
</div> </div>
{!! Datatable::table() <div style="min-height: 550px">
->addColumn( {!! Datatable::table()
trans('texts.date'), ->addColumn(
trans('texts.message'), trans('texts.date'),
trans('texts.balance'), trans('texts.message'),
trans('texts.adjustment')) trans('texts.balance'),
->setUrl(route('api.client.activity')) trans('texts.adjustment'))
->setOptions('bFilter', false) ->setUrl(route('api.client.activity'))
->setOptions('aaSorting', [['0', 'desc']]) ->setOptions('bFilter', false)
->setOptions('sPaginationType', 'bootstrap') ->setOptions('aaSorting', [['0', 'desc']])
->render('datatable') !!} ->setOptions('sPaginationType', 'bootstrap')
->render('datatable') !!}
</div>
</div> </div>