mirror of
https://github.com/solidtime-io/solidtime.git
synced 2026-08-08 08:12:17 +01:00
Compare commits
1 Commits
feature/po
...
feature/ba
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
d68c30476e |
3
.github/workflows/build-private.yml
vendored
3
.github/workflows/build-private.yml
vendored
@@ -10,8 +10,6 @@ on:
|
|||||||
- '.github/workflows/build-private.yml'
|
- '.github/workflows/build-private.yml'
|
||||||
- 'docker/prod/**'
|
- 'docker/prod/**'
|
||||||
workflow_dispatch:
|
workflow_dispatch:
|
||||||
permissions:
|
|
||||||
contents: read
|
|
||||||
|
|
||||||
name: Build - Private
|
name: Build - Private
|
||||||
jobs:
|
jobs:
|
||||||
@@ -19,7 +17,6 @@ jobs:
|
|||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
timeout-minutes: 20
|
timeout-minutes: 20
|
||||||
|
|
||||||
|
|
||||||
steps:
|
steps:
|
||||||
- name: "Check out code"
|
- name: "Check out code"
|
||||||
uses: actions/checkout@v4
|
uses: actions/checkout@v4
|
||||||
|
|||||||
16
.github/workflows/build-public.yml
vendored
16
.github/workflows/build-public.yml
vendored
@@ -11,12 +11,6 @@ on:
|
|||||||
- 'docker/prod/**'
|
- 'docker/prod/**'
|
||||||
workflow_dispatch:
|
workflow_dispatch:
|
||||||
|
|
||||||
permissions:
|
|
||||||
packages: write
|
|
||||||
contents: read
|
|
||||||
attestations: write
|
|
||||||
id-token: write
|
|
||||||
|
|
||||||
env:
|
env:
|
||||||
DOCKERHUB_REPO: solidtime/solidtime
|
DOCKERHUB_REPO: solidtime/solidtime
|
||||||
GHCR_REPO: ghcr.io/solidtime-io/solidtime
|
GHCR_REPO: ghcr.io/solidtime-io/solidtime
|
||||||
@@ -32,6 +26,11 @@ jobs:
|
|||||||
- runs-on: "ubuntu-24.04"
|
- runs-on: "ubuntu-24.04"
|
||||||
platform: "linux/amd64"
|
platform: "linux/amd64"
|
||||||
runs-on: ${{ matrix.runs-on }}
|
runs-on: ${{ matrix.runs-on }}
|
||||||
|
permissions:
|
||||||
|
packages: write
|
||||||
|
contents: read
|
||||||
|
attestations: write
|
||||||
|
id-token: write
|
||||||
timeout-minutes: 90
|
timeout-minutes: 90
|
||||||
|
|
||||||
steps:
|
steps:
|
||||||
@@ -164,6 +163,11 @@ jobs:
|
|||||||
|
|
||||||
merge:
|
merge:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
permissions:
|
||||||
|
packages: write
|
||||||
|
contents: read
|
||||||
|
attestations: write
|
||||||
|
id-token: write
|
||||||
timeout-minutes: 90
|
timeout-minutes: 90
|
||||||
needs:
|
needs:
|
||||||
- build
|
- build
|
||||||
|
|||||||
3
.github/workflows/generate-api-docs.yml
vendored
3
.github/workflows/generate-api-docs.yml
vendored
@@ -3,9 +3,6 @@ on:
|
|||||||
push:
|
push:
|
||||||
branches:
|
branches:
|
||||||
- main
|
- main
|
||||||
permissions:
|
|
||||||
contents: read
|
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
api_docs:
|
api_docs:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
|||||||
2
.github/workflows/npm-build.yml
vendored
2
.github/workflows/npm-build.yml
vendored
@@ -1,8 +1,6 @@
|
|||||||
name: NPM Build
|
name: NPM Build
|
||||||
|
|
||||||
on: [push]
|
on: [push]
|
||||||
permissions:
|
|
||||||
contents: read
|
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
build:
|
build:
|
||||||
|
|||||||
2
.github/workflows/npm-lint.yml
vendored
2
.github/workflows/npm-lint.yml
vendored
@@ -1,8 +1,6 @@
|
|||||||
name: NPM Lint
|
name: NPM Lint
|
||||||
|
|
||||||
on: [push]
|
on: [push]
|
||||||
permissions:
|
|
||||||
contents: read
|
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
build:
|
build:
|
||||||
|
|||||||
2
.github/workflows/npm-publish-api.yml
vendored
2
.github/workflows/npm-publish-api.yml
vendored
@@ -1,8 +1,6 @@
|
|||||||
name: Publish API package to NPM
|
name: Publish API package to NPM
|
||||||
on:
|
on:
|
||||||
workflow_dispatch
|
workflow_dispatch
|
||||||
permissions:
|
|
||||||
contents: read
|
|
||||||
jobs:
|
jobs:
|
||||||
build:
|
build:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
|||||||
2
.github/workflows/npm-publish-ui.yml
vendored
2
.github/workflows/npm-publish-ui.yml
vendored
@@ -1,8 +1,6 @@
|
|||||||
name: Publish UI package to NPM
|
name: Publish UI package to NPM
|
||||||
on:
|
on:
|
||||||
workflow_dispatch
|
workflow_dispatch
|
||||||
permissions:
|
|
||||||
contents: read
|
|
||||||
jobs:
|
jobs:
|
||||||
build:
|
build:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
|||||||
3
.github/workflows/npm-typecheck.yml
vendored
3
.github/workflows/npm-typecheck.yml
vendored
@@ -1,8 +1,7 @@
|
|||||||
name: NPM Typecheck
|
name: NPM Typecheck
|
||||||
|
|
||||||
on: [push]
|
on: [push]
|
||||||
permissions:
|
|
||||||
contents: read
|
|
||||||
jobs:
|
jobs:
|
||||||
build:
|
build:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
|||||||
2
.github/workflows/phpstan.yml
vendored
2
.github/workflows/phpstan.yml
vendored
@@ -1,7 +1,5 @@
|
|||||||
name: Static code analysis (PHPStan)
|
name: Static code analysis (PHPStan)
|
||||||
on: push
|
on: push
|
||||||
permissions:
|
|
||||||
contents: read
|
|
||||||
jobs:
|
jobs:
|
||||||
phpstan:
|
phpstan:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
|||||||
9
.github/workflows/phpunit.yml
vendored
9
.github/workflows/phpunit.yml
vendored
@@ -1,18 +1,13 @@
|
|||||||
name: PHPUnit Tests
|
name: PHPUnit Tests
|
||||||
on: push
|
on: push
|
||||||
permissions:
|
|
||||||
contents: read
|
|
||||||
jobs:
|
jobs:
|
||||||
phpunit:
|
phpunit:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
timeout-minutes: 10
|
timeout-minutes: 10
|
||||||
strategy:
|
|
||||||
matrix:
|
|
||||||
postgres_version: [ 15, 16, 17 ]
|
|
||||||
|
|
||||||
services:
|
services:
|
||||||
pgsql_test:
|
pgsql_test:
|
||||||
image: postgres:${{ matrix.postgres_version }}
|
image: postgres:15
|
||||||
env:
|
env:
|
||||||
PGPASSWORD: 'root'
|
PGPASSWORD: 'root'
|
||||||
POSTGRES_DB: 'laravel'
|
POSTGRES_DB: 'laravel'
|
||||||
@@ -68,7 +63,7 @@ jobs:
|
|||||||
run: php artisan test --stop-on-failure --coverage-text --coverage-clover=coverage.xml
|
run: php artisan test --stop-on-failure --coverage-text --coverage-clover=coverage.xml
|
||||||
|
|
||||||
- name: "Upload coverage reports to Codecov"
|
- name: "Upload coverage reports to Codecov"
|
||||||
uses: codecov/codecov-action@v5.4.3
|
uses: codecov/codecov-action@v5.4.2
|
||||||
with:
|
with:
|
||||||
token: ${{ secrets.CODECOV_TOKEN }}
|
token: ${{ secrets.CODECOV_TOKEN }}
|
||||||
slug: solidtime-io/solidtime
|
slug: solidtime-io/solidtime
|
||||||
|
|||||||
2
.github/workflows/pint.yml
vendored
2
.github/workflows/pint.yml
vendored
@@ -1,7 +1,5 @@
|
|||||||
name: PHP Linting
|
name: PHP Linting
|
||||||
on: push
|
on: push
|
||||||
permissions:
|
|
||||||
contents: read
|
|
||||||
jobs:
|
jobs:
|
||||||
pint:
|
pint:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
|||||||
2
.github/workflows/playwright.yml
vendored
2
.github/workflows/playwright.yml
vendored
@@ -1,7 +1,5 @@
|
|||||||
name: Playwright Tests
|
name: Playwright Tests
|
||||||
on: [push]
|
on: [push]
|
||||||
permissions:
|
|
||||||
contents: read
|
|
||||||
jobs:
|
jobs:
|
||||||
test:
|
test:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
|||||||
@@ -1,123 +0,0 @@
|
|||||||
<?php
|
|
||||||
|
|
||||||
declare(strict_types=1);
|
|
||||||
|
|
||||||
namespace App\Console\Commands\SelfHost;
|
|
||||||
|
|
||||||
use Illuminate\Console\Command;
|
|
||||||
use Illuminate\Database\Query\Builder;
|
|
||||||
use Illuminate\Database\Query\JoinClause;
|
|
||||||
use Illuminate\Support\Collection;
|
|
||||||
use Illuminate\Support\Facades\DB;
|
|
||||||
use Illuminate\Support\Facades\Log;
|
|
||||||
|
|
||||||
class SelfHostDatabaseConsistency extends Command
|
|
||||||
{
|
|
||||||
/**
|
|
||||||
* The name and signature of the console command.
|
|
||||||
*
|
|
||||||
* @var string
|
|
||||||
*/
|
|
||||||
protected $signature = 'self-host:database-consistency';
|
|
||||||
|
|
||||||
/**
|
|
||||||
* The console command description.
|
|
||||||
*
|
|
||||||
* @var string
|
|
||||||
*/
|
|
||||||
protected $description = '';
|
|
||||||
|
|
||||||
/**
|
|
||||||
* Execute the console command.
|
|
||||||
*/
|
|
||||||
public function handle(): int
|
|
||||||
{
|
|
||||||
$hadAProblem = false;
|
|
||||||
|
|
||||||
// Task need to be part of project in time entries
|
|
||||||
$problems = DB::table('time_entries')
|
|
||||||
->select(['time_entries.id as id'])
|
|
||||||
->join('tasks', 'time_entries.task_id', '=', 'tasks.id')
|
|
||||||
->where('tasks.project_id', '!=', DB::raw('time_entries.project_id'))
|
|
||||||
->get();
|
|
||||||
$this->logProblems($problems, 'Time entries have a task that does not belong to the project of the time entry', $hadAProblem);
|
|
||||||
|
|
||||||
// Client id is the client id of the project
|
|
||||||
$problems = DB::table('time_entries')
|
|
||||||
->select(['time_entries.id as id'])
|
|
||||||
->join('projects', 'time_entries.project_id', '=', 'projects.id')
|
|
||||||
->where(DB::raw('coalesce(projects.client_id::varchar, \'\')'), '!=', DB::raw('coalesce(time_entries.client_id::varchar, \'\')'))
|
|
||||||
->get();
|
|
||||||
$this->logProblems($problems, 'Time entries have a client that does not match the client of the project', $hadAProblem);
|
|
||||||
|
|
||||||
// Client id can only be not null if the project id is not null
|
|
||||||
$problems = DB::table('time_entries')
|
|
||||||
->select(['time_entries.id as id'])
|
|
||||||
->whereNotNull('client_id')
|
|
||||||
->whereNull('project_id')
|
|
||||||
->get();
|
|
||||||
$this->logProblems($problems, 'Time entries have a client but no project', $hadAProblem);
|
|
||||||
|
|
||||||
// Every user needs to be a member of at least one organization
|
|
||||||
$problems = DB::table('users')
|
|
||||||
->select(['users.id as id'])
|
|
||||||
->leftJoin('members', 'users.id', '=', 'members.user_id')
|
|
||||||
->whereNull('members.id')
|
|
||||||
->get();
|
|
||||||
$this->logProblems($problems, 'Users are not member of any organization', $hadAProblem);
|
|
||||||
|
|
||||||
// Every organization needs at least an owner
|
|
||||||
$problems = DB::table('organizations')
|
|
||||||
->select(['organizations.id as id'])
|
|
||||||
->leftJoin('members', function (JoinClause $join): void {
|
|
||||||
$join->on('organizations.id', '=', 'members.organization_id')
|
|
||||||
->where('members.role', '=', 'owner');
|
|
||||||
})
|
|
||||||
->whereNull('members.id')
|
|
||||||
->get();
|
|
||||||
$this->logProblems($problems, 'Organizations without an owner', $hadAProblem);
|
|
||||||
|
|
||||||
// Every member can only have one running time entry
|
|
||||||
$problems = DB::table('time_entries')
|
|
||||||
->select(['user_id as id'])
|
|
||||||
->whereNull('end')
|
|
||||||
->groupBy('user_id')
|
|
||||||
->havingRaw('count(*) > 1')
|
|
||||||
->get(['user_id', DB::raw('count(*) as count')]);
|
|
||||||
$this->logProblems($problems, 'Users with more than one running time entry', $hadAProblem);
|
|
||||||
|
|
||||||
// Users have a current organization that they are not a member of
|
|
||||||
$problems = DB::table('users')
|
|
||||||
->select(['users.id as id'])
|
|
||||||
->whereNotNull('current_team_id')
|
|
||||||
->whereNotIn('current_team_id', function (Builder $query): void {
|
|
||||||
$query->select('organization_id')
|
|
||||||
->from('members')
|
|
||||||
->whereColumn('members.user_id', 'users.id');
|
|
||||||
})->get();
|
|
||||||
$this->logProblems($problems, 'Users have a current organization that they are not a member of', $hadAProblem);
|
|
||||||
|
|
||||||
return $hadAProblem ? self::FAILURE : self::SUCCESS;
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @param Collection<int, \stdClass> $problems
|
|
||||||
*/
|
|
||||||
private function logProblems(Collection $problems, string $message, bool &$hadAProblem): void
|
|
||||||
{
|
|
||||||
$message = 'Consistency problem: '.$message;
|
|
||||||
if ($problems->isNotEmpty()) {
|
|
||||||
$ids = $problems->pluck('id');
|
|
||||||
$hadAProblem = true;
|
|
||||||
Log::error($message, [
|
|
||||||
'ids' => $ids,
|
|
||||||
]);
|
|
||||||
|
|
||||||
$error = $message;
|
|
||||||
foreach ($ids as $id) {
|
|
||||||
$error .= "\n - ".$id;
|
|
||||||
}
|
|
||||||
$this->error($error);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
@@ -25,10 +25,6 @@ class Kernel extends ConsoleKernel
|
|||||||
$schedule->command('self-host:telemetry')
|
$schedule->command('self-host:telemetry')
|
||||||
->when(fn (): bool => config('scheduling.tasks.self_hosting_telemetry'))
|
->when(fn (): bool => config('scheduling.tasks.self_hosting_telemetry'))
|
||||||
->twiceDaily();
|
->twiceDaily();
|
||||||
|
|
||||||
$schedule->command('self-host:database-consistency')
|
|
||||||
->when(fn (): bool => config('scheduling.tasks.self_hosting_database_consistency'))
|
|
||||||
->twiceDaily();
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|||||||
@@ -1,14 +0,0 @@
|
|||||||
<?php
|
|
||||||
|
|
||||||
declare(strict_types=1);
|
|
||||||
|
|
||||||
namespace App\Events;
|
|
||||||
|
|
||||||
use Illuminate\Foundation\Events\Dispatchable;
|
|
||||||
|
|
||||||
class DatabaseSeederAfterSeed
|
|
||||||
{
|
|
||||||
use Dispatchable;
|
|
||||||
|
|
||||||
public function __construct() {}
|
|
||||||
}
|
|
||||||
@@ -1,14 +0,0 @@
|
|||||||
<?php
|
|
||||||
|
|
||||||
declare(strict_types=1);
|
|
||||||
|
|
||||||
namespace App\Events;
|
|
||||||
|
|
||||||
use Illuminate\Foundation\Events\Dispatchable;
|
|
||||||
|
|
||||||
class DatabaseSeederBeforeDelete
|
|
||||||
{
|
|
||||||
use Dispatchable;
|
|
||||||
|
|
||||||
public function __construct() {}
|
|
||||||
}
|
|
||||||
@@ -23,7 +23,6 @@ use Filament\Tables;
|
|||||||
use Filament\Tables\Filters\TernaryFilter;
|
use Filament\Tables\Filters\TernaryFilter;
|
||||||
use Filament\Tables\Table;
|
use Filament\Tables\Table;
|
||||||
use Illuminate\Database\Eloquent\Builder;
|
use Illuminate\Database\Eloquent\Builder;
|
||||||
use Illuminate\Support\Collection;
|
|
||||||
use Illuminate\Support\Facades\Auth;
|
use Illuminate\Support\Facades\Auth;
|
||||||
use Illuminate\Support\Facades\Hash;
|
use Illuminate\Support\Facades\Hash;
|
||||||
use Korridor\LaravelModelValidationRules\Rules\UniqueEloquent;
|
use Korridor\LaravelModelValidationRules\Rules\UniqueEloquent;
|
||||||
@@ -208,14 +207,6 @@ class UserResource extends Resource
|
|||||||
}),
|
}),
|
||||||
])
|
])
|
||||||
->bulkActions([
|
->bulkActions([
|
||||||
Tables\Actions\BulkAction::make('Resend verification email')
|
|
||||||
->icon('heroicon-o-paper-airplane')
|
|
||||||
->action(function (Collection $records): void {
|
|
||||||
foreach ($records as $user) {
|
|
||||||
/** @var User $user */
|
|
||||||
$user->sendEmailVerificationNotification();
|
|
||||||
}
|
|
||||||
}),
|
|
||||||
]);
|
]);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -226,7 +226,6 @@ class TimeEntryController extends Controller
|
|||||||
'start' => $request->getStart()->timezone($timezone),
|
'start' => $request->getStart()->timezone($timezone),
|
||||||
'end' => $request->getEnd()->timezone($timezone),
|
'end' => $request->getEnd()->timezone($timezone),
|
||||||
'localization' => $localizationService,
|
'localization' => $localizationService,
|
||||||
'showBillableRate' => $showBillableRate,
|
|
||||||
]);
|
]);
|
||||||
$footerViewFile = file_get_contents(resource_path('views/reports/time-entry-index/pdf-footer.blade.php'));
|
$footerViewFile = file_get_contents(resource_path('views/reports/time-entry-index/pdf-footer.blade.php'));
|
||||||
if ($footerViewFile === false) {
|
if ($footerViewFile === false) {
|
||||||
@@ -429,7 +428,6 @@ class TimeEntryController extends Controller
|
|||||||
'end' => $request->getEnd()->timezone($timezone),
|
'end' => $request->getEnd()->timezone($timezone),
|
||||||
'debug' => $debug,
|
'debug' => $debug,
|
||||||
'localization' => $localizationService,
|
'localization' => $localizationService,
|
||||||
'showBillableRate' => $showBillableRate,
|
|
||||||
]);
|
]);
|
||||||
$footerViewFile = file_get_contents(resource_path('views/reports/time-entry-aggregate/pdf-footer.blade.php'));
|
$footerViewFile = file_get_contents(resource_path('views/reports/time-entry-aggregate/pdf-footer.blade.php'));
|
||||||
if ($footerViewFile === false) {
|
if ($footerViewFile === false) {
|
||||||
@@ -458,7 +456,7 @@ class TimeEntryController extends Controller
|
|||||||
->putFileAs($folderPath, new File($tempFolder->path($filenameTemp)), $filename);
|
->putFileAs($folderPath, new File($tempFolder->path($filenameTemp)), $filename);
|
||||||
} else {
|
} else {
|
||||||
Excel::store(
|
Excel::store(
|
||||||
new TimeEntriesReportExport($aggregatedData, $format, $currency, $group, $subGroup, $showBillableRate),
|
new TimeEntriesReportExport($aggregatedData, $format, $currency, $group, $subGroup),
|
||||||
$path,
|
$path,
|
||||||
config('filesystems.private'),
|
config('filesystems.private'),
|
||||||
$format->getExportPackageType(),
|
$format->getExportPackageType(),
|
||||||
|
|||||||
@@ -8,7 +8,9 @@ use Illuminate\Foundation\Http\FormRequest;
|
|||||||
|
|
||||||
class BaseFormRequest extends FormRequest
|
class BaseFormRequest extends FormRequest
|
||||||
{
|
{
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
* @param bool $bigInt
|
||||||
* @return list<string>
|
* @return list<string>
|
||||||
*/
|
*/
|
||||||
protected function moneyRules(bool $bigInt = false): array
|
protected function moneyRules(bool $bigInt = false): array
|
||||||
|
|||||||
@@ -100,18 +100,12 @@ class DeletionService
|
|||||||
|
|
||||||
// Make sure all users have at least one organization and delete placeholders
|
// Make sure all users have at least one organization and delete placeholders
|
||||||
foreach ($users as $user) {
|
foreach ($users as $user) {
|
||||||
/** @var User $user */
|
|
||||||
if ($ignoreUser !== null && $user->is($ignoreUser)) {
|
if ($ignoreUser !== null && $user->is($ignoreUser)) {
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
if ($user->is_placeholder) {
|
if ($user->is_placeholder) {
|
||||||
$user->delete();
|
$user->delete();
|
||||||
} else {
|
} else {
|
||||||
if ($user->current_team_id === $organization->getKey()) {
|
|
||||||
$user->currentOrganization()->disassociate();
|
|
||||||
$user->save();
|
|
||||||
}
|
|
||||||
|
|
||||||
$this->userService->makeSureUserHasAtLeastOneOrganization($user);
|
$this->userService->makeSureUserHasAtLeastOneOrganization($user);
|
||||||
$this->userService->makeSureUserHasCurrentOrganization($user);
|
$this->userService->makeSureUserHasCurrentOrganization($user);
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -164,11 +164,6 @@ class MemberService
|
|||||||
public function makeMemberToPlaceholder(Member $member, bool $makeSureUserHasAtLeastOneOrganization = true): void
|
public function makeMemberToPlaceholder(Member $member, bool $makeSureUserHasAtLeastOneOrganization = true): void
|
||||||
{
|
{
|
||||||
$user = $member->user;
|
$user = $member->user;
|
||||||
if ($user->current_team_id === $member->organization_id) {
|
|
||||||
$user->currentTeam()->disassociate();
|
|
||||||
$user->save();
|
|
||||||
}
|
|
||||||
|
|
||||||
$placeholderUser = $user->replicate();
|
$placeholderUser = $user->replicate();
|
||||||
$placeholderUser->is_placeholder = true;
|
$placeholderUser->is_placeholder = true;
|
||||||
$placeholderUser->save();
|
$placeholderUser->save();
|
||||||
@@ -180,7 +175,6 @@ class MemberService
|
|||||||
$this->userService->assignOrganizationEntitiesToDifferentUser($member->organization, $user, $placeholderUser);
|
$this->userService->assignOrganizationEntitiesToDifferentUser($member->organization, $user, $placeholderUser);
|
||||||
if ($makeSureUserHasAtLeastOneOrganization) {
|
if ($makeSureUserHasAtLeastOneOrganization) {
|
||||||
$this->userService->makeSureUserHasAtLeastOneOrganization($user);
|
$this->userService->makeSureUserHasAtLeastOneOrganization($user);
|
||||||
$this->userService->makeSureUserHasCurrentOrganization($user);
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -46,8 +46,6 @@ class TimeEntriesReportExport implements FromView, ShouldAutoSize, WithCustomCsv
|
|||||||
|
|
||||||
private TimeEntryAggregationType $subGroup;
|
private TimeEntryAggregationType $subGroup;
|
||||||
|
|
||||||
private bool $showBillableRate;
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @param array{
|
* @param array{
|
||||||
* grouped_type: string|null,
|
* grouped_type: string|null,
|
||||||
@@ -68,14 +66,13 @@ class TimeEntriesReportExport implements FromView, ShouldAutoSize, WithCustomCsv
|
|||||||
* cost: int|null
|
* cost: int|null
|
||||||
* } $data
|
* } $data
|
||||||
*/
|
*/
|
||||||
public function __construct(array $data, ExportFormat $exportFormat, string $currency, TimeEntryAggregationType $group, TimeEntryAggregationType $subGroup, bool $showBillableRate)
|
public function __construct(array $data, ExportFormat $exportFormat, string $currency, TimeEntryAggregationType $group, TimeEntryAggregationType $subGroup)
|
||||||
{
|
{
|
||||||
$this->data = $data;
|
$this->data = $data;
|
||||||
$this->exportFormat = $exportFormat;
|
$this->exportFormat = $exportFormat;
|
||||||
$this->currency = $currency;
|
$this->currency = $currency;
|
||||||
$this->group = $group;
|
$this->group = $group;
|
||||||
$this->subGroup = $subGroup;
|
$this->subGroup = $subGroup;
|
||||||
$this->showBillableRate = $showBillableRate;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
public function view(): View
|
public function view(): View
|
||||||
@@ -86,7 +83,6 @@ class TimeEntriesReportExport implements FromView, ShouldAutoSize, WithCustomCsv
|
|||||||
'group' => $this->group,
|
'group' => $this->group,
|
||||||
'subGroup' => $this->subGroup,
|
'subGroup' => $this->subGroup,
|
||||||
'exportFormat' => $this->exportFormat,
|
'exportFormat' => $this->exportFormat,
|
||||||
'showBillableRate' => $this->showBillableRate,
|
|
||||||
]);
|
]);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -114,15 +114,13 @@ class UserService
|
|||||||
|
|
||||||
public function makeSureUserHasCurrentOrganization(User $user): void
|
public function makeSureUserHasCurrentOrganization(User $user): void
|
||||||
{
|
{
|
||||||
if ($user->current_team_id !== null) {
|
if ($user->currentOrganization !== null) {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
$organization = $user->organizations()->first();
|
$organization = $user->organizations()->first();
|
||||||
if ($organization !== null) {
|
$user->currentOrganization()->associate($organization);
|
||||||
$user->currentOrganization()->associate($organization);
|
$user->save();
|
||||||
$user->save();
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|||||||
@@ -8,6 +8,5 @@ return [
|
|||||||
'time_entry_send_still_running_mails' => (bool) env('SCHEDULING_TASK_TIME_ENTRY_SEND_STILL_RUNNING_MAILS', true),
|
'time_entry_send_still_running_mails' => (bool) env('SCHEDULING_TASK_TIME_ENTRY_SEND_STILL_RUNNING_MAILS', true),
|
||||||
'self_hosting_check_for_update' => (bool) env('SCHEDULING_TASK_SELF_HOSTING_CHECK_FOR_UPDATE', true),
|
'self_hosting_check_for_update' => (bool) env('SCHEDULING_TASK_SELF_HOSTING_CHECK_FOR_UPDATE', true),
|
||||||
'self_hosting_telemetry' => (bool) env('SCHEDULING_TASK_SELF_HOSTING_TELEMETRY', true),
|
'self_hosting_telemetry' => (bool) env('SCHEDULING_TASK_SELF_HOSTING_TELEMETRY', true),
|
||||||
'self_hosting_database_consistency' => (bool) env('SCHEDULING_TASK_SELF_HOSTING_DATABASE_CONSISTENCY', false),
|
|
||||||
],
|
],
|
||||||
];
|
];
|
||||||
|
|||||||
@@ -1,43 +0,0 @@
|
|||||||
<?php
|
|
||||||
|
|
||||||
declare(strict_types=1);
|
|
||||||
|
|
||||||
use Illuminate\Database\Migrations\Migration;
|
|
||||||
use Illuminate\Database\Schema\Blueprint;
|
|
||||||
use Illuminate\Support\Facades\Schema;
|
|
||||||
|
|
||||||
return new class extends Migration
|
|
||||||
{
|
|
||||||
/**
|
|
||||||
* Run the migrations.
|
|
||||||
*/
|
|
||||||
public function up(): void
|
|
||||||
{
|
|
||||||
DB::statement('
|
|
||||||
update users
|
|
||||||
set current_team_id = null
|
|
||||||
where id in (
|
|
||||||
select users.id from users
|
|
||||||
left join organizations on users.current_team_id = organizations.id
|
|
||||||
where users.current_team_id is not null and organizations.id is null
|
|
||||||
)
|
|
||||||
');
|
|
||||||
Schema::table('users', function (Blueprint $table): void {
|
|
||||||
$table->foreign('current_team_id', 'organizations_current_organization_id_foreign')
|
|
||||||
->references('id')
|
|
||||||
->on('organizations')
|
|
||||||
->onDelete('restrict')
|
|
||||||
->onUpdate('cascade');
|
|
||||||
});
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* Reverse the migrations.
|
|
||||||
*/
|
|
||||||
public function down(): void
|
|
||||||
{
|
|
||||||
Schema::table('users', function (Blueprint $table): void {
|
|
||||||
$table->dropForeign('organizations_current_organization_id_foreign');
|
|
||||||
});
|
|
||||||
}
|
|
||||||
};
|
|
||||||
@@ -5,8 +5,6 @@ declare(strict_types=1);
|
|||||||
namespace Database\Seeders;
|
namespace Database\Seeders;
|
||||||
|
|
||||||
use App\Enums\Role;
|
use App\Enums\Role;
|
||||||
use App\Events\DatabaseSeederAfterSeed;
|
|
||||||
use App\Events\DatabaseSeederBeforeDelete;
|
|
||||||
use App\Models\Audit;
|
use App\Models\Audit;
|
||||||
use App\Models\Client;
|
use App\Models\Client;
|
||||||
use App\Models\Member;
|
use App\Models\Member;
|
||||||
@@ -186,13 +184,10 @@ class DatabaseSeeder extends Seeder
|
|||||||
'email' => 'admin@example.com',
|
'email' => 'admin@example.com',
|
||||||
]);
|
]);
|
||||||
|
|
||||||
DatabaseSeederAfterSeed::dispatch();
|
|
||||||
}
|
}
|
||||||
|
|
||||||
private function deleteAll(): void
|
private function deleteAll(): void
|
||||||
{
|
{
|
||||||
DatabaseSeederBeforeDelete::dispatch();
|
|
||||||
|
|
||||||
// Laravel Passport tables
|
// Laravel Passport tables
|
||||||
DB::table((new RefreshToken)->getTable())->delete();
|
DB::table((new RefreshToken)->getTable())->delete();
|
||||||
DB::table((new Token)->getTable())->delete();
|
DB::table((new Token)->getTable())->delete();
|
||||||
@@ -218,9 +213,6 @@ class DatabaseSeeder extends Seeder
|
|||||||
DB::table((new Client)->getTable())->delete();
|
DB::table((new Client)->getTable())->delete();
|
||||||
DB::table((new Member)->getTable())->delete();
|
DB::table((new Member)->getTable())->delete();
|
||||||
DB::table((new OrganizationInvitation)->getTable())->delete();
|
DB::table((new OrganizationInvitation)->getTable())->delete();
|
||||||
DB::table((new User)->getTable())->update([
|
|
||||||
'current_team_id' => null,
|
|
||||||
]);
|
|
||||||
DB::table((new Organization)->getTable())->delete();
|
DB::table((new Organization)->getTable())->delete();
|
||||||
DB::table((new User)->getTable())->delete();
|
DB::table((new User)->getTable())->delete();
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -9,8 +9,7 @@ return [
|
|||||||
'2. In the same preferences page change the language of Clockfiy to English.<br>'.
|
'2. In the same preferences page change the language of Clockfiy to English.<br>'.
|
||||||
'3. Go to REPORTS -> TIME -> Detailed in the navigation on the left. <br>'.
|
'3. Go to REPORTS -> TIME -> Detailed in the navigation on the left. <br>'.
|
||||||
'4. Now select the date range that you want to export in the right top. '.
|
'4. Now select the date range that you want to export in the right top. '.
|
||||||
'In the free Clockify plan it\'s currently not possible to select more than one year. '.
|
'It is currently not possible to select more than one year. You can export each year separately and import them one after another .'.
|
||||||
'You can export each year separately and import them one after another.'.
|
|
||||||
'<br> 4. Now click Export -> Save as CSV. The Export dropdown is in the header of the export table left of the printer symbol. '.
|
'<br> 4. Now click Export -> Save as CSV. The Export dropdown is in the header of the export table left of the printer symbol. '.
|
||||||
'<br><br>Before you import make sure that the Timezone settings in Clockify are the same as in solidtime.',
|
'<br><br>Before you import make sure that the Timezone settings in Clockify are the same as in solidtime.',
|
||||||
],
|
],
|
||||||
|
|||||||
@@ -44,7 +44,7 @@
|
|||||||
--theme-color-input-select-active: rgb(var(--color-accent-300));
|
--theme-color-input-select-active: rgb(var(--color-accent-300));
|
||||||
--theme-color-input-select-active-hover: rgb(var(--color-accent-200));
|
--theme-color-input-select-active-hover: rgb(var(--color-accent-200));
|
||||||
|
|
||||||
--color-accent-default: rgba(var(--color-accent-300), 0.2);
|
--color-accent-default: rgb(var(--color-accent-900));
|
||||||
--color-accent-foreground: rgb(var(--color-accent-100));
|
--color-accent-foreground: rgb(var(--color-accent-100));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -5,7 +5,6 @@ import { h, ref } from 'vue';
|
|||||||
import type { CreateReportBodyProperties } from '@/packages/api/src';
|
import type { CreateReportBodyProperties } from '@/packages/api/src';
|
||||||
import { isAllowedToPerformPremiumAction } from '@/utils/billing';
|
import { isAllowedToPerformPremiumAction } from '@/utils/billing';
|
||||||
import UpgradeModal from '@/Components/Common/UpgradeModal.vue';
|
import UpgradeModal from '@/Components/Common/UpgradeModal.vue';
|
||||||
import { canCreateReports } from '@/utils/permissions';
|
|
||||||
defineProps<{
|
defineProps<{
|
||||||
reportProperties: CreateReportBodyProperties;
|
reportProperties: CreateReportBodyProperties;
|
||||||
}>();
|
}>();
|
||||||
@@ -34,10 +33,7 @@ function onSaveReportClick() {
|
|||||||
<strong>Sharable Reports</strong> is only available in solidtime
|
<strong>Sharable Reports</strong> is only available in solidtime
|
||||||
Professional.
|
Professional.
|
||||||
</UpgradeModal>
|
</UpgradeModal>
|
||||||
<SecondaryButton
|
<SecondaryButton :icon="SaveIcon" @click="onSaveReportClick"
|
||||||
v-if="canCreateReports()"
|
|
||||||
:icon="SaveIcon"
|
|
||||||
@click="onSaveReportClick"
|
|
||||||
>Save Report</SecondaryButton
|
>Save Report</SecondaryButton
|
||||||
>
|
>
|
||||||
</template>
|
</template>
|
||||||
|
|||||||
@@ -107,10 +107,6 @@ function getFilterAttributes(): AggregatedTimeEntriesQueryParams {
|
|||||||
: undefined,
|
: undefined,
|
||||||
tag_ids: selectedTags.value.length > 0 ? selectedTags.value : undefined,
|
tag_ids: selectedTags.value.length > 0 ? selectedTags.value : undefined,
|
||||||
billable: billable.value !== null ? billable.value : undefined,
|
billable: billable.value !== null ? billable.value : undefined,
|
||||||
member_id:
|
|
||||||
getCurrentRole() === 'employee'
|
|
||||||
? getCurrentMembershipId()
|
|
||||||
: undefined,
|
|
||||||
};
|
};
|
||||||
return params;
|
return params;
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -34,18 +34,11 @@ const forwarded = useForwardPropsEmits(delegatedProps, emits)
|
|||||||
v-bind="forwarded"
|
v-bind="forwarded"
|
||||||
:class="
|
:class="
|
||||||
cn(
|
cn(
|
||||||
'fixed top-0 left-0 z-50 w-screen h-screen flex items-start pt-6 md:pt-20 xl:pt-32 justify-center overflow-auto data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0 data-[state=closed]:zoom-out-95 data-[state=open]:zoom-in-95',
|
'fixed left-1/2 top-1/3 bg-default-background z-50 grid w-full max-w-lg -translate-x-1/2 -translate-y-1/2 border shadow-lg duration-200 data-[state=open]:animate-in data-[state=closed]:animate-out data-[state=closed]:fade-out-0 data-[state=open]:fade-in-0 data-[state=closed]:zoom-out-95 data-[state=open]:zoom-in-95 data-[state=closed]:slide-out-to-left-1/2 data-[state=closed]:slide-out-to-top-[48%] data-[state=open]:slide-in-from-left-1/2 data-[state=open]:slide-in-from-top-[48%] sm:rounded-lg',
|
||||||
|
|
||||||
)"
|
|
||||||
>
|
|
||||||
<div
|
|
||||||
:class="cn(
|
|
||||||
'bg-default-background grid w-full max-w-lg border shadow-lg duration-200 sm:rounded-lg',
|
|
||||||
props.class,
|
props.class,
|
||||||
)"
|
)"
|
||||||
>
|
>
|
||||||
<slot />
|
<slot />
|
||||||
</div>
|
|
||||||
</DialogContent>
|
</DialogContent>
|
||||||
</DialogPortal>
|
</DialogPortal>
|
||||||
</template>
|
</template>
|
||||||
|
|||||||
@@ -1,7 +1,7 @@
|
|||||||
<script setup lang="ts">
|
<script setup lang="ts">
|
||||||
import FormSection from '@/Components/FormSection.vue';
|
import FormSection from '@/Components/FormSection.vue';
|
||||||
import PrimaryButton from '@/packages/ui/src/Buttons/PrimaryButton.vue';
|
import PrimaryButton from '@/packages/ui/src/Buttons/PrimaryButton.vue';
|
||||||
import {computed, ref, inject, type ComputedRef} from 'vue';
|
import {computed, ref} from 'vue';
|
||||||
import InputLabel from '@/packages/ui/src/Input/InputLabel.vue';
|
import InputLabel from '@/packages/ui/src/Input/InputLabel.vue';
|
||||||
import {
|
import {
|
||||||
api,
|
api,
|
||||||
@@ -23,7 +23,6 @@ import {useNotificationsStore} from "@/utils/notification";
|
|||||||
import {useClipboard} from "@vueuse/core";
|
import {useClipboard} from "@vueuse/core";
|
||||||
import { formatDateTimeLocalized} from "../../../packages/ui/src/utils/time";
|
import { formatDateTimeLocalized} from "../../../packages/ui/src/utils/time";
|
||||||
import {ClockIcon} from "@heroicons/vue/20/solid";
|
import {ClockIcon} from "@heroicons/vue/20/solid";
|
||||||
import type { Organization } from '@/packages/api/src';
|
|
||||||
|
|
||||||
const queryClient = useQueryClient();
|
const queryClient = useQueryClient();
|
||||||
|
|
||||||
@@ -35,8 +34,6 @@ const newToken = ref('');
|
|||||||
|
|
||||||
const { copy, copied, isSupported } = useClipboard();
|
const { copy, copied, isSupported } = useClipboard();
|
||||||
|
|
||||||
const organization = inject<ComputedRef<Organization>>('organization');
|
|
||||||
|
|
||||||
async function createApiToken(){
|
async function createApiToken(){
|
||||||
await handleApiRequestNotifications(
|
await handleApiRequestNotifications(
|
||||||
() =>
|
() =>
|
||||||
@@ -216,10 +213,10 @@ const revokeApiTokenMutation = useMutation({
|
|||||||
<div>{{ token.name }}</div>
|
<div>{{ token.name }}</div>
|
||||||
<div class="text-sm text-text-tertiary space-x-3">
|
<div class="text-sm text-text-tertiary space-x-3">
|
||||||
<span v-if="token.created_at">
|
<span v-if="token.created_at">
|
||||||
Created at {{ formatDateTimeLocalized(token.created_at, organization?.date_format, organization?.time_format) }}
|
Created at {{ formatDateTimeLocalized(token.created_at) }}
|
||||||
</span>
|
</span>
|
||||||
<span v-if="token.expires_at">
|
<span v-if="token.expires_at">
|
||||||
Expires at {{ formatDateTimeLocalized(token.expires_at, organization?.date_format, organization?.time_format) }}
|
Expires at {{ formatDateTimeLocalized(token.expires_at) }}
|
||||||
</span>
|
</span>
|
||||||
<span v-if="token.revoked">
|
<span v-if="token.revoked">
|
||||||
Revoked
|
Revoked
|
||||||
|
|||||||
@@ -160,7 +160,10 @@ const tableData = computed(() => {
|
|||||||
cost: el.cost,
|
cost: el.cost,
|
||||||
description:
|
description:
|
||||||
el.description ??
|
el.description ??
|
||||||
emptyPlaceholder[entry.grouped_type ?? 'project'],
|
emptyPlaceholder[
|
||||||
|
aggregatedTableTimeEntries.value
|
||||||
|
?.grouped_type ?? 'project'
|
||||||
|
],
|
||||||
};
|
};
|
||||||
}) ?? [],
|
}) ?? [],
|
||||||
};
|
};
|
||||||
|
|||||||
@@ -5,7 +5,10 @@ import {
|
|||||||
PopoverTrigger,
|
PopoverTrigger,
|
||||||
} from '@/Components/ui/popover';
|
} from '@/Components/ui/popover';
|
||||||
import { RangeCalendar } from '@/Components/ui/range-calendar';
|
import { RangeCalendar } from '@/Components/ui/range-calendar';
|
||||||
import { CalendarDate } from '@internationalized/date';
|
import {
|
||||||
|
CalendarDate,
|
||||||
|
getLocalTimeZone,
|
||||||
|
} from '@internationalized/date';
|
||||||
import { CalendarIcon } from 'lucide-vue-next';
|
import { CalendarIcon } from 'lucide-vue-next';
|
||||||
import { computed, ref, inject, type ComputedRef, watch } from 'vue';
|
import { computed, ref, inject, type ComputedRef, watch } from 'vue';
|
||||||
import { twMerge } from 'tailwind-merge';
|
import { twMerge } from 'tailwind-merge';
|
||||||
@@ -13,9 +16,8 @@ import {
|
|||||||
getDayJsInstance,
|
getDayJsInstance,
|
||||||
getLocalizedDayJs,
|
getLocalizedDayJs,
|
||||||
} from '@/packages/ui/src/utils/time';
|
} from '@/packages/ui/src/utils/time';
|
||||||
|
import { formatDateLocalized } from '@/packages/ui/src/utils/time';
|
||||||
import { type Organization } from '@/packages/api/src';
|
import { type Organization } from '@/packages/api/src';
|
||||||
import { getUserTimezone } from '@/packages/ui/src/utils/settings';
|
|
||||||
import { formatDate } from '@/packages/ui/src/utils/time';
|
|
||||||
|
|
||||||
const props = defineProps<{
|
const props = defineProps<{
|
||||||
start: string;
|
start: string;
|
||||||
@@ -57,13 +59,12 @@ const modelValue = computed<CalendarDateRange>({
|
|||||||
}),
|
}),
|
||||||
set: (newValue) => {
|
set: (newValue) => {
|
||||||
if (newValue.start) {
|
if (newValue.start) {
|
||||||
console.log(newValue.start);
|
const date = newValue.start.toDate(getLocalTimeZone());
|
||||||
const date = newValue.start.toDate(getUserTimezone());
|
emit('update:start', getDayJsInstance()(date).format('YYYY-MM-DD'));
|
||||||
emit('update:start', getLocalizedDayJs(date.toString()).format());
|
|
||||||
}
|
}
|
||||||
if (newValue.end) {
|
if (newValue.end) {
|
||||||
const date = newValue.end.toDate(getUserTimezone());
|
const date = newValue.end.toDate(getLocalTimeZone());
|
||||||
emit('update:end', getLocalizedDayJs(date.toString()).format());
|
emit('update:end', getDayJsInstance()(date).format('YYYY-MM-DD'));
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
});
|
});
|
||||||
@@ -218,27 +219,12 @@ watch(open, (value) => {
|
|||||||
<CalendarIcon class="mr-2 h-4 w-4" />
|
<CalendarIcon class="mr-2 h-4 w-4" />
|
||||||
<template v-if="modelValue.start">
|
<template v-if="modelValue.start">
|
||||||
<template v-if="modelValue.end">
|
<template v-if="modelValue.end">
|
||||||
{{
|
{{ formatDateLocalized(modelValue.start.toString(), organization?.date_format) }}
|
||||||
formatDate(
|
|
||||||
modelValue.start.toString(),
|
|
||||||
organization?.date_format
|
|
||||||
)
|
|
||||||
}}
|
|
||||||
-
|
-
|
||||||
{{
|
{{ formatDateLocalized(modelValue.end.toString(), organization?.date_format) }}
|
||||||
formatDate(
|
|
||||||
modelValue.end.toString(),
|
|
||||||
organization?.date_format
|
|
||||||
)
|
|
||||||
}}
|
|
||||||
</template>
|
</template>
|
||||||
<template v-else>
|
<template v-else>
|
||||||
{{
|
{{ formatDateLocalized(modelValue.start.toString(), organization?.date_format) }}
|
||||||
formatDate(
|
|
||||||
modelValue.start.toString(),
|
|
||||||
organization?.date_format
|
|
||||||
)
|
|
||||||
}}
|
|
||||||
</template>
|
</template>
|
||||||
</template>
|
</template>
|
||||||
<template v-else> Pick a date </template>
|
<template v-else> Pick a date </template>
|
||||||
|
|||||||
@@ -146,9 +146,8 @@ export function formatDateLocalized(date: string, format: DateFormat = 'point-se
|
|||||||
return getLocalizedDayJs(date).format(dateFormatMap[format]);
|
return getLocalizedDayJs(date).format(dateFormatMap[format]);
|
||||||
}
|
}
|
||||||
|
|
||||||
export function formatDateTimeLocalized(date: string, dateFormat?: DateFormat, timeFormat?: TimeFormat): string {
|
export function formatDateTimeLocalized(date: string): string {
|
||||||
const format = `${dateFormatMap[dateFormat ?? 'point-separated-d-m-yyyy']} ${timeFormat === '12-hours' ? 'hh:mm A' : 'HH:mm'}`;
|
return getLocalizedDayJs(date).format('DD.MM.YYYY HH:mm');
|
||||||
return getLocalizedDayJs(date).format(format);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
export function formatWeek(date: string | null): string {
|
export function formatWeek(date: string | null): string {
|
||||||
|
|||||||
@@ -125,6 +125,4 @@ export function canViewAllTimeEntries() {
|
|||||||
export function canViewInvoices() {
|
export function canViewInvoices() {
|
||||||
return currentUserHasPermission('invoices:view');
|
return currentUserHasPermission('invoices:view');
|
||||||
}
|
}
|
||||||
export function canCreateReports() {
|
|
||||||
return currentUserHasPermission('reports:create');
|
|
||||||
}
|
|
||||||
|
|||||||
@@ -6,11 +6,7 @@ import type {
|
|||||||
AggregatedTimeEntriesQueryParams,
|
AggregatedTimeEntriesQueryParams,
|
||||||
ReportingResponse,
|
ReportingResponse,
|
||||||
} from '@/packages/api/src';
|
} from '@/packages/api/src';
|
||||||
import {
|
import { getCurrentOrganizationId } from '@/utils/useUser';
|
||||||
getCurrentOrganizationId,
|
|
||||||
getCurrentRole,
|
|
||||||
getCurrentUser,
|
|
||||||
} from '@/utils/useUser';
|
|
||||||
import { useNotificationsStore } from '@/utils/notification';
|
import { useNotificationsStore } from '@/utils/notification';
|
||||||
import { useProjectsStore } from '@/utils/useProjects';
|
import { useProjectsStore } from '@/utils/useProjects';
|
||||||
import { useMembersStore } from '@/utils/useMembers';
|
import { useMembersStore } from '@/utils/useMembers';
|
||||||
@@ -110,9 +106,6 @@ export const useReportingStore = defineStore('reporting', () => {
|
|||||||
return projects.value.find((project) => project.id === key)?.name;
|
return projects.value.find((project) => project.id === key)?.name;
|
||||||
}
|
}
|
||||||
if (type === 'user') {
|
if (type === 'user') {
|
||||||
if (getCurrentRole() === 'employee') {
|
|
||||||
return getCurrentUser().name;
|
|
||||||
}
|
|
||||||
const memberStore = useMembersStore();
|
const memberStore = useMembersStore();
|
||||||
const { members } = storeToRefs(memberStore);
|
const { members } = storeToRefs(memberStore);
|
||||||
return members.value.find((member) => member.user_id === key)?.name;
|
return members.value.find((member) => member.user_id === key)?.name;
|
||||||
|
|||||||
@@ -10,10 +10,6 @@ function getCurrentUserId() {
|
|||||||
return page.props.auth.user.id;
|
return page.props.auth.user.id;
|
||||||
}
|
}
|
||||||
|
|
||||||
function getCurrentUser() {
|
|
||||||
return page.props.auth.user;
|
|
||||||
}
|
|
||||||
|
|
||||||
function getCurrentOrganizationId() {
|
function getCurrentOrganizationId() {
|
||||||
return page.props.auth.user.current_team_id;
|
return page.props.auth.user.current_team_id;
|
||||||
}
|
}
|
||||||
@@ -35,5 +31,4 @@ export {
|
|||||||
getCurrentUserId,
|
getCurrentUserId,
|
||||||
getCurrentMembershipId,
|
getCurrentMembershipId,
|
||||||
getCurrentRole,
|
getCurrentRole,
|
||||||
getCurrentUser,
|
|
||||||
};
|
};
|
||||||
|
|||||||
@@ -152,13 +152,12 @@
|
|||||||
<div
|
<div
|
||||||
style="font-size: 24px; font-weight: 500; margin-top: 2px;">{{ $localization->formatInterval(CarbonInterval::seconds($aggregatedData['seconds'])) }} </div>
|
style="font-size: 24px; font-weight: 500; margin-top: 2px;">{{ $localization->formatInterval(CarbonInterval::seconds($aggregatedData['seconds'])) }} </div>
|
||||||
</div>
|
</div>
|
||||||
@if($showBillableRate)
|
|
||||||
<div style="padding: 8px 12px; border-radius: 8px;">
|
<div style="padding: 8px 12px; border-radius: 8px;">
|
||||||
<div style="color: #71717a; font-weight: 600;">Total cost</div>
|
<div style="color: #71717a; font-weight: 600;">Total cost</div>
|
||||||
<div
|
<div
|
||||||
style="font-size: 24px; font-weight: 500; margin-top: 2px;">{{ $localization->formatCurrency(Money::of(BigDecimal::ofUnscaledValue($aggregatedData['cost'], 2)->__toString(), $currency)) }} </div>
|
style="font-size: 24px; font-weight: 500; margin-top: 2px;">{{ $localization->formatCurrency(Money::of(BigDecimal::ofUnscaledValue($aggregatedData['cost'], 2)->__toString(), $currency)) }} </div>
|
||||||
</div>
|
</div>
|
||||||
@endif
|
|
||||||
</div>
|
</div>
|
||||||
<div id="main-chart" style="width: 700px; height: 300px; margin: 20px auto;"></div>
|
<div id="main-chart" style="width: 700px; height: 300px; margin: 20px auto;"></div>
|
||||||
|
|
||||||
@@ -178,9 +177,7 @@
|
|||||||
{{ $group->description() }}
|
{{ $group->description() }}
|
||||||
</th>
|
</th>
|
||||||
<th>Duration</th>
|
<th>Duration</th>
|
||||||
@if($showBillableRate)
|
|
||||||
<th style="text-align: right;">Cost</th>
|
<th style="text-align: right;">Cost</th>
|
||||||
@endif
|
|
||||||
</tr>
|
</tr>
|
||||||
</thead>
|
</thead>
|
||||||
@foreach($aggregatedData['grouped_data'] as $group1Entry)
|
@foreach($aggregatedData['grouped_data'] as $group1Entry)
|
||||||
@@ -191,21 +188,23 @@
|
|||||||
}};">
|
}};">
|
||||||
</div>
|
</div>
|
||||||
<span style="padding-left: 8px;">
|
<span style="padding-left: 8px;">
|
||||||
|
|
||||||
@if($group->is(\App\Enums\TimeEntryAggregationType::Billable))
|
@if($group->is(\App\Enums\TimeEntryAggregationType::Billable))
|
||||||
{{ $group1Entry['key'] === '1' ? 'Billable' : 'Non-billable' }}
|
{{ $group1Entry['key'] === '1' ? 'Billable' : 'Non-billable' }}
|
||||||
@else
|
@else
|
||||||
{{ $group1Entry['description'] ?? $group1Entry['key'] ?? 'No '.Str::lower($group->description()) }}
|
{{ $group1Entry['description'] ?? $group1Entry['key'] ?? 'No '.Str::lower($group->description()) }}
|
||||||
@endif
|
@endif
|
||||||
</span>
|
|
||||||
|
|
||||||
|
</span>
|
||||||
</td>
|
</td>
|
||||||
<td style="text-align: left;">
|
<td style="text-align: left;">
|
||||||
{{ $localization->formatInterval(CarbonInterval::seconds($group1Entry['seconds'])) }}
|
{{ $localization->formatInterval(CarbonInterval::seconds($group1Entry['seconds'])) }}
|
||||||
</td>
|
</td>
|
||||||
@if($showBillableRate)
|
|
||||||
<td style="text-align: right;">
|
<td style="text-align: right;">
|
||||||
{{ $localization->formatCurrency(Money::of(BigDecimal::ofUnscaledValue($group1Entry['cost'], 2)->__toString(), $currency)) }}
|
{{ $localization->formatCurrency(Money::of(BigDecimal::ofUnscaledValue($group1Entry['cost'], 2)->__toString(), $currency)) }}
|
||||||
</td>
|
</td>
|
||||||
@endif
|
|
||||||
</tr>
|
</tr>
|
||||||
@endforeach
|
@endforeach
|
||||||
<tfoot>
|
<tfoot>
|
||||||
@@ -216,11 +215,9 @@
|
|||||||
<td style="font-weight: 500;color: #18181b;">
|
<td style="font-weight: 500;color: #18181b;">
|
||||||
{{ $localization->formatInterval(CarbonInterval::seconds($aggregatedData['seconds'])) }}
|
{{ $localization->formatInterval(CarbonInterval::seconds($aggregatedData['seconds'])) }}
|
||||||
</td>
|
</td>
|
||||||
@if($showBillableRate)
|
|
||||||
<td style="text-align: right; font-weight: 500;color: #18181b;">
|
<td style="text-align: right; font-weight: 500;color: #18181b;">
|
||||||
{{ $localization->formatCurrency(Money::of(BigDecimal::ofUnscaledValue($aggregatedData['cost'], 2)->__toString(), $currency)) }}
|
{{ $localization->formatCurrency(Money::of(BigDecimal::ofUnscaledValue($aggregatedData['cost'], 2)->__toString(), $currency)) }}
|
||||||
</td>
|
</td>
|
||||||
@endif
|
|
||||||
</tr>
|
</tr>
|
||||||
</tfoot>
|
</tfoot>
|
||||||
</table>
|
</table>
|
||||||
@@ -256,11 +253,9 @@
|
|||||||
<th>
|
<th>
|
||||||
Duration (h)
|
Duration (h)
|
||||||
</th>
|
</th>
|
||||||
@if($showBillableRate)
|
|
||||||
<th>
|
<th>
|
||||||
Cost
|
Cost
|
||||||
</th>
|
</th>
|
||||||
@endif
|
|
||||||
</tr>
|
</tr>
|
||||||
</thead>
|
</thead>
|
||||||
<tbody>
|
<tbody>
|
||||||
@@ -287,17 +282,13 @@
|
|||||||
<td>
|
<td>
|
||||||
{{ $localization->formatNumber($duration->totalHours) }}
|
{{ $localization->formatNumber($duration->totalHours) }}
|
||||||
</td>
|
</td>
|
||||||
@if($showBillableRate)
|
|
||||||
<td>
|
<td>
|
||||||
{{ $localization->formatCurrency(Money::of(BigDecimal::ofUnscaledValue($group2Entry['cost'], 2)->__toString(), $currency)) }}
|
{{ $localization->formatCurrency(Money::of(BigDecimal::ofUnscaledValue($group2Entry['cost'], 2)->__toString(), $currency)) }}
|
||||||
</td>
|
</td>
|
||||||
@endif
|
|
||||||
</tr>
|
</tr>
|
||||||
@php
|
@php
|
||||||
$totalDuration += $group2Entry['seconds'];
|
$totalDuration += $group2Entry['seconds'];
|
||||||
if ($showBillableRate) {
|
$totalCost += $group2Entry['cost'];
|
||||||
$totalCost += $group2Entry['cost'];
|
|
||||||
}
|
|
||||||
@endphp
|
@endphp
|
||||||
@endforeach
|
@endforeach
|
||||||
</tbody>
|
</tbody>
|
||||||
|
|||||||
@@ -62,11 +62,9 @@
|
|||||||
<td style="border: 1px solid black;" data-type="{{ DataType::TYPE_STRING }}">
|
<td style="border: 1px solid black;" data-type="{{ DataType::TYPE_STRING }}">
|
||||||
{{ round($duration->totalHours, 2) }}
|
{{ round($duration->totalHours, 2) }}
|
||||||
</td>
|
</td>
|
||||||
@if($showBillableRate)
|
|
||||||
<td style="border: 1px solid black;" data-type="{{ DataType::TYPE_STRING }}">
|
<td style="border: 1px solid black;" data-type="{{ DataType::TYPE_STRING }}">
|
||||||
{{ round(BigDecimal::ofUnscaledValue($group2Entry['cost'], 2)->toFloat(), 2) }}
|
{{ round(BigDecimal::ofUnscaledValue($group2Entry['cost'], 2)->toFloat(), 2) }}
|
||||||
</td>
|
</td>
|
||||||
@endif
|
|
||||||
@else
|
@else
|
||||||
@if ($group === TimeEntryAggregationType::Billable)
|
@if ($group === TimeEntryAggregationType::Billable)
|
||||||
<td style="border: 1px solid black;" data-type="{{ DataType::TYPE_STRING }}">
|
<td style="border: 1px solid black;" data-type="{{ DataType::TYPE_STRING }}">
|
||||||
@@ -94,20 +92,16 @@
|
|||||||
data-format="{{ NumberFormat::FORMAT_NUMBER_00 }}">
|
data-format="{{ NumberFormat::FORMAT_NUMBER_00 }}">
|
||||||
{{ $duration->totalHours }}
|
{{ $duration->totalHours }}
|
||||||
</td>
|
</td>
|
||||||
@if($showBillableRate)
|
|
||||||
<td style="border: 1px solid black;" data-type="{{ DataType::TYPE_NUMERIC }}"
|
<td style="border: 1px solid black;" data-type="{{ DataType::TYPE_NUMERIC }}"
|
||||||
data-format="{{ NumberFormat::FORMAT_NUMBER_COMMA_SEPARATED1 }}">
|
data-format="{{ NumberFormat::FORMAT_NUMBER_COMMA_SEPARATED1 }}">
|
||||||
{{ BigDecimal::ofUnscaledValue($group2Entry['cost'], 2)->__toString() }}
|
{{ BigDecimal::ofUnscaledValue($group2Entry['cost'], 2)->__toString() }}
|
||||||
</td>
|
</td>
|
||||||
@endif
|
|
||||||
@endif
|
@endif
|
||||||
</tr>
|
</tr>
|
||||||
@php
|
@php
|
||||||
++$counter;
|
++$counter;
|
||||||
$totalDuration += $group2Entry['seconds'];
|
$totalDuration += $group2Entry['seconds'];
|
||||||
if ($showBillableRate) {
|
$totalCost += $group2Entry['cost'];
|
||||||
$totalCost += $group2Entry['cost'];
|
|
||||||
}
|
|
||||||
@endphp
|
@endphp
|
||||||
@endforeach
|
@endforeach
|
||||||
@endforeach
|
@endforeach
|
||||||
@@ -126,11 +120,9 @@
|
|||||||
<td style="border: 1px solid black; font-weight: bold;" data-type="{{ DataType::TYPE_STRING }}">
|
<td style="border: 1px solid black; font-weight: bold;" data-type="{{ DataType::TYPE_STRING }}">
|
||||||
{{ round($totalDurationInterval->totalHours, 2) }}
|
{{ round($totalDurationInterval->totalHours, 2) }}
|
||||||
</td>
|
</td>
|
||||||
@if($showBillableRate)
|
|
||||||
<td style="border: 1px solid black; font-weight: bold;" data-type="{{ DataType::TYPE_STRING }}">
|
<td style="border: 1px solid black; font-weight: bold;" data-type="{{ DataType::TYPE_STRING }}">
|
||||||
{{ round(BigDecimal::ofUnscaledValue($totalCost, 2)->toFloat(), 2) }}
|
{{ round(BigDecimal::ofUnscaledValue($totalCost, 2)->toFloat(), 2) }}
|
||||||
</td>
|
</td>
|
||||||
@endif
|
|
||||||
@else
|
@else
|
||||||
<td style="border: 1px solid black; font-weight: bold;" data-type="{{ DataType::TYPE_FORMULA }}"
|
<td style="border: 1px solid black; font-weight: bold;" data-type="{{ DataType::TYPE_FORMULA }}"
|
||||||
data-format="[hh]:mm:ss">
|
data-format="[hh]:mm:ss">
|
||||||
|
|||||||
@@ -140,14 +140,12 @@
|
|||||||
<div
|
<div
|
||||||
style="font-size: 24px; font-weight: 500; margin-top: 2px;">{{ $localization->formatInterval(CarbonInterval::seconds($aggregatedData['seconds'])) }} </div>
|
style="font-size: 24px; font-weight: 500; margin-top: 2px;">{{ $localization->formatInterval(CarbonInterval::seconds($aggregatedData['seconds'])) }} </div>
|
||||||
</div>
|
</div>
|
||||||
@if($showBillableRate)
|
|
||||||
<div style="padding: 8px 12px; border-radius: 8px;">
|
<div style="padding: 8px 12px; border-radius: 8px;">
|
||||||
<div style="color: #71717a; font-weight: 600;">Total cost</div>
|
<div style="color: #71717a; font-weight: 600;">Total cost</div>
|
||||||
<div style="font-size: 24px; font-weight: 500; margin-top: 2px;">
|
<div
|
||||||
{{ $localization->formatCurrency(Money::of(BigDecimal::ofUnscaledValue($aggregatedData['cost'], 2)->__toString(), $currency)) }}
|
style="font-size: 24px; font-weight: 500; margin-top: 2px;">{{ $localization->formatCurrency(Money::of(BigDecimal::ofUnscaledValue($aggregatedData['cost'], 2)->__toString(), $currency)) }} </div>
|
||||||
</div>
|
|
||||||
</div>
|
</div>
|
||||||
@endif
|
|
||||||
</div>
|
</div>
|
||||||
<div>
|
<div>
|
||||||
<table style="width: 100%;">
|
<table style="width: 100%;">
|
||||||
|
|||||||
@@ -56,12 +56,10 @@ abstract class TestCaseWithDatabase extends TestCase
|
|||||||
/**
|
/**
|
||||||
* @return object{user: User, organization: Organization, member: Member, owner: User, ownerMember: Member}
|
* @return object{user: User, organization: Organization, member: Member, owner: User, ownerMember: Member}
|
||||||
*/
|
*/
|
||||||
public function createUserWithRole(Role $role, bool $employeesCanSeeBillableRates = false): object
|
public function createUserWithRole(Role $role): object
|
||||||
{
|
{
|
||||||
$owner = User::factory()->create();
|
$owner = User::factory()->create();
|
||||||
$organization = Organization::factory()->withOwner($owner)->create([
|
$organization = Organization::factory()->withOwner($owner)->create();
|
||||||
'employees_can_see_billable_rates' => $employeesCanSeeBillableRates,
|
|
||||||
]);
|
|
||||||
$ownerMember = Member::factory()->forUser($owner)->forOrganization($organization)->role(Role::Owner)->create();
|
$ownerMember = Member::factory()->forUser($owner)->forOrganization($organization)->role(Role::Owner)->create();
|
||||||
$owner->currentOrganization()->associate($organization);
|
$owner->currentOrganization()->associate($organization);
|
||||||
$owner->save();
|
$owner->save();
|
||||||
|
|||||||
@@ -1,163 +0,0 @@
|
|||||||
<?php
|
|
||||||
|
|
||||||
declare(strict_types=1);
|
|
||||||
|
|
||||||
namespace Tests\Unit\Console\Commands\SelfHost;
|
|
||||||
|
|
||||||
use App\Console\Commands\SelfHost\SelfHostDatabaseConsistency;
|
|
||||||
use App\Enums\Role;
|
|
||||||
use App\Models\Client;
|
|
||||||
use App\Models\Organization;
|
|
||||||
use App\Models\Project;
|
|
||||||
use App\Models\Task;
|
|
||||||
use App\Models\TimeEntry;
|
|
||||||
use App\Models\User;
|
|
||||||
use Illuminate\Console\Command;
|
|
||||||
use Illuminate\Support\Facades\Artisan;
|
|
||||||
use PHPUnit\Framework\Attributes\CoversClass;
|
|
||||||
use PHPUnit\Framework\Attributes\UsesClass;
|
|
||||||
use Tests\TestCaseWithDatabase;
|
|
||||||
|
|
||||||
#[CoversClass(SelfHostDatabaseConsistency::class)]
|
|
||||||
#[UsesClass(SelfHostDatabaseConsistency::class)]
|
|
||||||
class SelfHostDatabaseConsistencyCommandTest extends TestCaseWithDatabase
|
|
||||||
{
|
|
||||||
public function test_checks_that_task_need_to_be_part_of_project_in_time_entries(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$user = $this->createUserWithRole(Role::Owner);
|
|
||||||
$project1 = Project::factory()->forOrganization($user->organization)->create();
|
|
||||||
$project2 = Project::factory()->forOrganization($user->organization)->create();
|
|
||||||
$task = Task::factory()->forOrganization($user->organization)->forProject($project1)->create();
|
|
||||||
$timeEntry = TimeEntry::factory()->forMember($user->member)->forTask($task)->forProject($project2)->create();
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$exitCode = $this->withoutMockingConsoleOutput()->artisan('self-host:database-consistency');
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertSame(Command::FAILURE, $exitCode);
|
|
||||||
$output = Artisan::output();
|
|
||||||
$this->assertSame("Consistency problem: Time entries have a task that does not belong to the project of the time entry\n - ".$timeEntry->getKey()."\n", $output);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_checks_that_client_id_is_the_client_id_of_the_project(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$user = $this->createUserWithRole(Role::Owner);
|
|
||||||
$client1 = Client::factory()->forOrganization($user->organization)->create();
|
|
||||||
$client2 = Client::factory()->forOrganization($user->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($user->organization)->forClient($client1)->create();
|
|
||||||
$timeEntry = TimeEntry::factory()->forMember($user->member)->forProject($project)->create([
|
|
||||||
'client_id' => $client2->id,
|
|
||||||
]);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$exitCode = $this->withoutMockingConsoleOutput()->artisan('self-host:database-consistency');
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertSame(Command::FAILURE, $exitCode);
|
|
||||||
$output = Artisan::output();
|
|
||||||
$this->assertSame("Consistency problem: Time entries have a client that does not match the client of the project\n - ".$timeEntry->getKey()."\n", $output);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_checks_that_client_id_is_the_client_id_of_the_project_with_no_client_in_time_entry(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$user = $this->createUserWithRole(Role::Owner);
|
|
||||||
$client1 = Client::factory()->forOrganization($user->organization)->create();
|
|
||||||
$client2 = Client::factory()->forOrganization($user->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($user->organization)->forClient($client1)->create();
|
|
||||||
$timeEntry = TimeEntry::factory()->forMember($user->member)->forProject($project)->create([
|
|
||||||
'client_id' => null,
|
|
||||||
]);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$exitCode = $this->withoutMockingConsoleOutput()->artisan('self-host:database-consistency');
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertSame(Command::FAILURE, $exitCode);
|
|
||||||
$output = Artisan::output();
|
|
||||||
$this->assertSame("Consistency problem: Time entries have a client that does not match the client of the project\n - ".$timeEntry->getKey()."\n", $output);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_checks_that_client_id_is_only_null_if_project_is_also_null(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$user = $this->createUserWithRole(Role::Owner);
|
|
||||||
$client1 = Client::factory()->forOrganization($user->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($user->organization)->forClient($client1)->create();
|
|
||||||
$timeEntry = TimeEntry::factory()->forMember($user->member)->create([
|
|
||||||
'client_id' => $client1->getKey(),
|
|
||||||
]);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$exitCode = $this->withoutMockingConsoleOutput()->artisan('self-host:database-consistency');
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertSame(Command::FAILURE, $exitCode);
|
|
||||||
$output = Artisan::output();
|
|
||||||
$this->assertSame("Consistency problem: Time entries have a client but no project\n - ".$timeEntry->getKey()."\n", $output);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_checks_that_every_user_needs_to_be_a_member_of_at_least_one_organization(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$user = User::factory()->create();
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$exitCode = $this->withoutMockingConsoleOutput()->artisan('self-host:database-consistency');
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertSame(Command::FAILURE, $exitCode);
|
|
||||||
$output = Artisan::output();
|
|
||||||
$this->assertSame("Consistency problem: Users are not member of any organization\n - ".$user->getKey()."\n", $output);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_checks_that_every_organization_needs_at_least_an_owner(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$user = $this->createUserWithRole(Role::Owner);
|
|
||||||
$organization = Organization::factory()->withOwner($user->user)->create();
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$exitCode = $this->withoutMockingConsoleOutput()->artisan('self-host:database-consistency');
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertSame(Command::FAILURE, $exitCode);
|
|
||||||
$output = Artisan::output();
|
|
||||||
$this->assertSame("Consistency problem: Organizations without an owner\n - ".$organization->getKey()."\n", $output);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_checks_that_every_member_can_only_have_one_running_time_entry(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$user = $this->createUserWithRole(Role::Owner);
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forMember($user->member)->active()->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forMember($user->member)->active()->create();
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$exitCode = $this->withoutMockingConsoleOutput()->artisan('self-host:database-consistency');
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertSame(Command::FAILURE, $exitCode);
|
|
||||||
$output = Artisan::output();
|
|
||||||
$this->assertSame("Consistency problem: Users with more than one running time entry\n - ".$user->user->getKey()."\n", $output);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_checks_that_users_have_a_current_organization_that_they_are_not_a_member_of(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$user1 = $this->createUserWithRole(Role::Owner);
|
|
||||||
$user2 = $this->createUserWithRole(Role::Owner);
|
|
||||||
$user1->user->currentOrganization()->associate($user2->organization);
|
|
||||||
$user1->user->save();
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$exitCode = $this->withoutMockingConsoleOutput()->artisan('self-host:database-consistency');
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertSame(Command::FAILURE, $exitCode);
|
|
||||||
$output = Artisan::output();
|
|
||||||
$this->assertSame("Consistency problem: Users have a current organization that they are not a member of\n - ".$user1->user->getKey()."\n", $output);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
@@ -686,10 +686,6 @@ class TimeEntryEndpointTest extends ApiEndpointTestAbstract
|
|||||||
'time-entries:view:all',
|
'time-entries:view:all',
|
||||||
]);
|
]);
|
||||||
Passport::actingAs($data->user);
|
Passport::actingAs($data->user);
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$this->actAsOrganizationWithSubscription();
|
$this->actAsOrganizationWithSubscription();
|
||||||
|
|
||||||
// Act
|
// Act
|
||||||
@@ -704,192 +700,6 @@ class TimeEntryEndpointTest extends ApiEndpointTestAbstract
|
|||||||
$this->assertResponseCode($response, 200);
|
$this->assertResponseCode($response, 200);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function test_index_export_endpoint_can_create_a_detailed_time_entry_report_in_format_csv_as_employee_role_with_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, true);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.index-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::CSV,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->id,
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_index_export_endpoint_can_create_a_detailed_time_entry_report_in_format_ods_as_employee_role_with_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, true);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.index-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::ODS,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->id,
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_index_export_endpoint_can_create_a_detailed_time_entry_report_in_format_xlxs_as_employee_role_with_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, true);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.index-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::XLSX,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->id,
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_index_export_endpoint_can_create_a_detailed_time_entry_report_in_format_pdf_as_employee_role_with_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, true);
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$this->actAsOrganizationWithSubscription();
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.index-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::PDF,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->id,
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_index_export_endpoint_can_create_a_detailed_time_entry_report_in_format_csv_as_employee_role_without_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, false);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.index-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::CSV,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->id,
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_index_export_endpoint_can_create_a_detailed_time_entry_report_in_format_ods_as_employee_role_without_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, false);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.index-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::ODS,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->id,
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_index_export_endpoint_can_create_a_detailed_time_entry_report_in_format_xlxs_as_employee_role_without_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, false);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.index-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::XLSX,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->id,
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_index_export_endpoint_can_create_a_detailed_time_entry_report_in_format_pdf_as_employee_role_without_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, false);
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$this->actAsOrganizationWithSubscription();
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.index-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::PDF,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->id,
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_aggregate_export_endpoints_fails_if_user_no_permission_to_view_time_entries(): void
|
public function test_aggregate_export_endpoints_fails_if_user_no_permission_to_view_time_entries(): void
|
||||||
{
|
{
|
||||||
// Arrange
|
// Arrange
|
||||||
@@ -1005,58 +815,6 @@ class TimeEntryEndpointTest extends ApiEndpointTestAbstract
|
|||||||
$this->assertResponseCode($response, 200);
|
$this->assertResponseCode($response, 200);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function test_aggregate_export_endpoints_can_create_a_csv_report_as_employee_role_with_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, true);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.aggregate-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::CSV,
|
|
||||||
'group' => TimeEntryAggregationType::Client,
|
|
||||||
'sub_group' => TimeEntryAggregationType::Project,
|
|
||||||
'history_group' => TimeEntryAggregationTypeInterval::Month,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->getKey(),
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_aggregate_export_endpoints_can_create_a_csv_report_as_employee_role_without_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, false);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.aggregate-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::CSV,
|
|
||||||
'group' => TimeEntryAggregationType::Client,
|
|
||||||
'sub_group' => TimeEntryAggregationType::Project,
|
|
||||||
'history_group' => TimeEntryAggregationTypeInterval::Month,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->getKey(),
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_aggregate_export_endpoints_can_create_a_xlsx_report(): void
|
public function test_aggregate_export_endpoints_can_create_a_xlsx_report(): void
|
||||||
{
|
{
|
||||||
// Arrange
|
// Arrange
|
||||||
@@ -1084,58 +842,6 @@ class TimeEntryEndpointTest extends ApiEndpointTestAbstract
|
|||||||
$this->assertResponseCode($response, 200);
|
$this->assertResponseCode($response, 200);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function test_aggregate_export_endpoints_can_create_a_xlsx_report_as_employee_role_with_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, true);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.aggregate-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::XLSX,
|
|
||||||
'group' => TimeEntryAggregationType::Client,
|
|
||||||
'sub_group' => TimeEntryAggregationType::Project,
|
|
||||||
'history_group' => TimeEntryAggregationTypeInterval::Month,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->getKey(),
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_aggregate_export_endpoints_can_create_a_xlsx_report_as_employee_role_without_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, false);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.aggregate-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::XLSX,
|
|
||||||
'group' => TimeEntryAggregationType::Client,
|
|
||||||
'sub_group' => TimeEntryAggregationType::Project,
|
|
||||||
'history_group' => TimeEntryAggregationTypeInterval::Month,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->getKey(),
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_aggregate_export_endpoints_can_create_a_ods_report(): void
|
public function test_aggregate_export_endpoints_can_create_a_ods_report(): void
|
||||||
{
|
{
|
||||||
// Arrange
|
// Arrange
|
||||||
@@ -1163,58 +869,6 @@ class TimeEntryEndpointTest extends ApiEndpointTestAbstract
|
|||||||
$this->assertResponseCode($response, 200);
|
$this->assertResponseCode($response, 200);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function test_aggregate_export_endpoints_can_create_a_ods_report_as_employee_role_with_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, true);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.aggregate-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::ODS,
|
|
||||||
'group' => TimeEntryAggregationType::User,
|
|
||||||
'sub_group' => TimeEntryAggregationType::Project,
|
|
||||||
'history_group' => TimeEntryAggregationTypeInterval::Month,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->getKey(),
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_aggregate_export_endpoints_can_create_a_ods_report_as_employee_role_without_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, false);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.aggregate-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::ODS,
|
|
||||||
'group' => TimeEntryAggregationType::User,
|
|
||||||
'sub_group' => TimeEntryAggregationType::Project,
|
|
||||||
'history_group' => TimeEntryAggregationTypeInterval::Month,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->getKey(),
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_aggregate_export_endpoint_fails_if_pdf_renderer_is_not_configured_but_a_user_want_a_pdf_report(): void
|
public function test_aggregate_export_endpoint_fails_if_pdf_renderer_is_not_configured_but_a_user_want_a_pdf_report(): void
|
||||||
{
|
{
|
||||||
// Arrange
|
// Arrange
|
||||||
@@ -1273,60 +927,6 @@ class TimeEntryEndpointTest extends ApiEndpointTestAbstract
|
|||||||
$this->assertResponseCode($response, 200);
|
$this->assertResponseCode($response, 200);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function test_aggregate_export_endpoints_can_create_a_pdf_report_as_employee_role_with_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, true);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
$this->actAsOrganizationWithSubscription();
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.aggregate-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::PDF,
|
|
||||||
'group' => TimeEntryAggregationType::User,
|
|
||||||
'sub_group' => TimeEntryAggregationType::Project,
|
|
||||||
'history_group' => TimeEntryAggregationTypeInterval::Month,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->getKey(),
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_aggregate_export_endpoints_can_create_a_pdf_report_as_employee_role_without_show_billable_rate(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$data = $this->createUserWithRole(Role::Employee, false);
|
|
||||||
$client = Client::factory()->forOrganization($data->organization)->create();
|
|
||||||
$project = Project::factory()->forOrganization($data->organization)->forClient($client)->create();
|
|
||||||
$timeEntry1 = TimeEntry::factory()->forOrganization($data->organization)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
$timeEntry2 = TimeEntry::factory()->forOrganization($data->organization)->forProject($project)->forMember($data->member)->startWithDuration(Carbon::now(), 100)->create();
|
|
||||||
Passport::actingAs($data->user);
|
|
||||||
$this->actAsOrganizationWithSubscription();
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$response = $this->getJson(route('api.v1.time-entries.aggregate-export', [
|
|
||||||
$data->organization->getKey(),
|
|
||||||
'format' => ExportFormat::PDF,
|
|
||||||
'group' => TimeEntryAggregationType::User,
|
|
||||||
'sub_group' => TimeEntryAggregationType::Project,
|
|
||||||
'history_group' => TimeEntryAggregationTypeInterval::Month,
|
|
||||||
'start' => Carbon::now()->startOfYear()->toIso8601ZuluString(),
|
|
||||||
'end' => Carbon::now()->endOfYear()->toIso8601ZuluString(),
|
|
||||||
'member_id' => $data->member->getKey(),
|
|
||||||
]));
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertResponseCode($response, 200);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_aggregate_endpoint_fails_if_user_has_only_access_to_own_time_entries_but_does_not_filter_for_this(): void
|
public function test_aggregate_endpoint_fails_if_user_has_only_access_to_own_time_entries_but_does_not_filter_for_this(): void
|
||||||
{
|
{
|
||||||
// Arrange
|
// Arrange
|
||||||
|
|||||||
@@ -150,24 +150,6 @@ class DeletionServiceTest extends TestCaseWithDatabase
|
|||||||
$this->assertSame($specialCase ? 7 : 6, TimeEntry::query()->whereBelongsTo($organization, 'organization')->count());
|
$this->assertSame($specialCase ? 7 : 6, TimeEntry::query()->whereBelongsTo($organization, 'organization')->count());
|
||||||
}
|
}
|
||||||
|
|
||||||
public function test_delete_organization_resets_the_current_organization_of_users_that_had_the_deleted_organization_as_current_organization(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$userOwner = User::factory()->create();
|
|
||||||
$organization = Organization::factory()->withOwner($userOwner)->create();
|
|
||||||
$userOwner->currentOrganization()->associate($organization);
|
|
||||||
$userOwner->save();
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$this->deletionService->deleteOrganization($organization);
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$this->assertOrganizationDeleted($organization);
|
|
||||||
$userOwner->refresh();
|
|
||||||
$this->assertNull($userOwner->current_team_id);
|
|
||||||
$this->assertNotSame($organization->id, $userOwner->current_team_id);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_delete_organization_deletes_all_resources_of_the_organization_but_does_not_delete_other_resources(): void
|
public function test_delete_organization_deletes_all_resources_of_the_organization_but_does_not_delete_other_resources(): void
|
||||||
{
|
{
|
||||||
// Arrange
|
// Arrange
|
||||||
|
|||||||
@@ -114,41 +114,6 @@ class MemberServiceTest extends TestCaseWithDatabase
|
|||||||
$this->assertSame(1, $otherUser->organizations()->count());
|
$this->assertSame(1, $otherUser->organizations()->count());
|
||||||
}
|
}
|
||||||
|
|
||||||
public function test_make_member_to_placeholder_resets_current_organization_of_user_if_user_is_no_longer_member_to_newly_created_organization(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$organization = Organization::factory()->create();
|
|
||||||
$user = User::factory()->forCurrentOrganization($organization)->create();
|
|
||||||
$member = Member::factory()->forOrganization($organization)->forUser($user)->role(Role::Employee)->create();
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$this->memberService->makeMemberToPlaceholder($member);
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$user->refresh();
|
|
||||||
$this->assertNotNull($user->current_team_id);
|
|
||||||
$this->assertNotSame($organization->id, $user->current_team_id);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_make_member_to_placeholder_resets_current_organization_of_user_if_user_is_no_longer_member_to_already_existing_other_organization(): void
|
|
||||||
{
|
|
||||||
// Arrange
|
|
||||||
$organization = Organization::factory()->create();
|
|
||||||
$user = User::factory()->forCurrentOrganization($organization)->create();
|
|
||||||
$member = Member::factory()->forOrganization($organization)->forUser($user)->role(Role::Employee)->create();
|
|
||||||
|
|
||||||
$otherOrganization = Organization::factory()->create();
|
|
||||||
$otherMember = Member::factory()->forOrganization($otherOrganization)->forUser($user)->role(Role::Employee)->create();
|
|
||||||
|
|
||||||
// Act
|
|
||||||
$this->memberService->makeMemberToPlaceholder($member);
|
|
||||||
|
|
||||||
// Assert
|
|
||||||
$user->refresh();
|
|
||||||
$this->assertNotNull($user->current_team_id);
|
|
||||||
$this->assertSame($otherOrganization->id, $user->current_team_id);
|
|
||||||
}
|
|
||||||
|
|
||||||
public function test_assign_organization_entities_to_different_member_without_any_entries(): void
|
public function test_assign_organization_entities_to_different_member_without_any_entries(): void
|
||||||
{
|
{
|
||||||
// Arrange
|
// Arrange
|
||||||
|
|||||||
Reference in New Issue
Block a user