Fix database migrations for mailgateways

CHANGELOG.md

@@ -1,21 +1,5 @@
 # Letzte Änderungen
 
-### 1.12.6
-
--   Fix: Beiträge von Familienmitgliedern splitten
-
-### 1.12.5
-
--   Fix: Synchronisieren von bestehenden Beiträgen aus NaMi
-
-### 1.12.4
-
--   Filter Mitglieder nach Verhaltenskodex
-
-### 1.12.3
-
--   Volltextsuche für Rechnungen
-
 ### 1.12.2
 
 -   Zuschussliste Gallier

app/Actions/InsertMemberAction.php

@@ -22,8 +22,7 @@ class InsertMemberAction
     {
         $region = Region::firstWhere('nami_id', $member->regionId ?: -1);
 
-
-        $payload = [
+        return Member::updateOrCreate(['nami_id' => $member->id], [
             'firstname' => $member->firstname,
             'lastname' => $member->lastname,
             'joined_at' => $member->joinedAt,
@@ -52,16 +51,7 @@ class InsertMemberAction
             'mitgliedsnr' => $member->memberId,
             'version' => $member->version,
             'keepdata' => $member->keepdata,
-        ];
-
-        // Dont update subscription if fee id of existing member's subscription is already the same
-        if ($existing = Member::nami($member->id)) {
-            if ($existing->subscription && $existing->subscription->fee->nami_id === $member->feeId) {
-                $payload['subscription_id'] = $existing->subscription->id;
-            }
-        }
-
-        return Member::updateOrCreate(['nami_id' => $member->id], $payload);
+        ]);
     }
 
     public function getSubscription(NamiMember $member): ?Subscription

app/Invoice/Models/Invoice.php

@@ -49,6 +49,7 @@ class Invoice extends Model
      */
     public static function createForMember(Member $member, Collection $members, int $year, Subscription $subscription = null): self
     {
+        $subscription = $subscription ?: $member->subscription;
         $invoice = new self([
             'to' => [
                 'name' => 'Familie ' . $member->lastname,
@@ -65,8 +66,7 @@ class Invoice extends Model
 
         $positions = collect([]);
         foreach ($members as $member) {
-            $memberSubscription = $subscription ?: $member->subscription;
-            foreach ($memberSubscription->children as $child) {
+            foreach ($subscription->children as $child) {
                 $positions->push([
                     'description' => str($child->name)->replace('{name}', $member->firstname . ' ' . $member->lastname)->replace('{year}', (string) $year),
                     'price' => $child->amount,

app/Member/FilterScope.php

@@ -46,8 +46,6 @@ class FilterScope extends ScoutFilter
         public array $exclude = [],
         public ?bool $hasFullAddress = null,
         public ?bool $hasBirthday = null,
-        public ?bool $hasSvk = null,
-        public ?bool $hasVk = null,
     ) {
     }
 
@@ -87,12 +85,6 @@ class FilterScope extends ScoutFilter
             if ($this->hasBirthday === true) {
                 $filter->push('birthday IS NOT NULL');
             }
-            if ($this->hasSvk !== null) {
-                $filter->push('has_svk = ' . ($this->hasSvk ? 'true' : 'false'));
-            }
-            if ($this->hasVk !== null) {
-                $filter->push('has_vk = ' . ($this->hasVk ? 'true' : 'false'));
-            }
             if ($this->ausstand === true) {
                 $filter->push('ausstand > 0');
             }

app/Member/Member.php

@@ -544,8 +544,6 @@ class Member extends Model implements Geolocatable
             'bill_kind' => $this->bill_kind?->value,
             'group_id' => $this->group->id,
             'group_name' => $this->group->inner_name ?: $this->group->name,
-            'has_vk' => $this->has_vk,
-            'has_svk' => $this->has_svk,
             'links' => [
                 'show' => route('member.show', ['member' => $this], false),
                 'edit' => route('member.edit', ['member' => $this], false),

app/Member/MemberResource.php

@@ -163,10 +163,6 @@ class MemberResource extends JsonResource
                 'activity_ids' => [],
                 'subactivity_ids' => []
             ],
-            'boolean_filter' => [
-                ['id' => true, 'name' => 'Ja'],
-                ['id' => false, 'name' => 'Nein'],
-            ],
             'default' => [
                 'gender_id' => null,
                 'salutation' => '',

config/scout.php

@@ -138,7 +138,7 @@ return [
         'key' => env('MEILI_MASTER_KEY', null),
         'index-settings' => [
             Member::class => [
-                'filterableAttributes' => ['address', 'birthday', 'ausstand', 'bill_kind', 'group_id', 'memberships', 'has_vk', 'has_svk', 'id'],
+                'filterableAttributes' => ['address', 'birthday', 'ausstand', 'bill_kind', 'group_id', 'memberships', 'id'],
                 'searchableAttributes' => ['fullname', 'address'],
                 'sortableAttributes' => ['lastname', 'firstname'],
                 'displayedAttributes' => ['age_group_icon', 'group_name', 'links', 'is_leader', 'lastname', 'firstname', 'fullname', 'address', 'ausstand', 'birthday', 'id', 'memberships', 'bill_kind', 'group_id'],

database/factories/Member/MemberFactory.php

@@ -37,8 +37,6 @@ class MemberFactory extends Factory
             'email' => $this->faker->safeEmail(),
             'recertified_at' => null,
             'keepdata' => false,
-            'has_svk' => $this->faker->boolean(),
-            'has_vk' => $this->faker->boolean(),
         ];
     }
 

resources/js/views/member/VIndex.vue

@@ -48,24 +48,6 @@
                     size="sm"
                     @update:model-value="setFilter('ausstand', $event)"
                 ></f-switch>
-                <f-select
-                    id="has_vk"
-                    name="has_vk"
-                    :model-value="getFilter('has_vk')"
-                    label="Verhaltenskodex unterschrieben"
-                    size="sm"
-                    :options="meta.boolean_filter"
-                    @update:model-value="setFilter('has_vk', $event)"
-                ></f-select>
-                <f-select
-                    id="has_svk"
-                    name="has_svk"
-                    :model-value="getFilter('has_svk')"
-                    label="SVK unterschrieben"
-                    size="sm"
-                    :options="meta.boolean_filter"
-                    @update:model-value="setFilter('has_svk', $event)"
-                ></f-select>
                 <f-multipleselect
                     id="group_ids"
                     :options="meta.groups"

tests/EndToEnd/MemberIndexTest.php

@@ -202,30 +202,6 @@ class MemberIndexTest extends EndToEndTestCase
         ]])->assertInertiaCount('data.data', 1);
     }
 
-    public function testItFiltersForSvkPrevention(): void
-    {
-        Member::factory()->defaults()->create(['has_svk' => true]);
-        Member::factory()->defaults()->create(['has_svk' => false]);
-        Member::factory()->defaults()->create(['has_svk' => false]);
-
-        sleep(1);
-        $this->callFilter('member.index', ['has_svk' => true])->assertInertiaCount('data.data', 1);
-        $this->callFilter('member.index', ['has_svk' => false])->assertInertiaCount('data.data', 2);
-        $this->callFilter('member.index', ['has_svk' => null])->assertInertiaCount('data.data', 3);
-    }
-
-    public function testItFiltersForVkPrevention(): void
-    {
-        Member::factory()->defaults()->create(['has_vk' => true]);
-        Member::factory()->defaults()->create(['has_vk' => false]);
-        Member::factory()->defaults()->create(['has_vk' => false]);
-
-        sleep(1);
-        $this->callFilter('member.index', ['has_vk' => true])->assertInertiaCount('data.data', 1);
-        $this->callFilter('member.index', ['has_vk' => false])->assertInertiaCount('data.data', 2);
-        $this->callFilter('member.index', ['has_vk' => null])->assertInertiaCount('data.data', 3);
-    }
-
     public function testGroupOfMembershipsFilterCanBeEmpty(): void
     {
         $mitglied = Activity::factory()->create();

tests/Feature/Invoice/MassStoreActionTest.php

@@ -90,8 +90,8 @@ class MassStoreActionTest extends TestCase
 
         $this->assertDatabaseCount('invoices', 1);
         $this->assertDatabaseCount('invoice_positions', 2);
-        $this->assertDatabaseHas('invoice_positions', ['description' => 'beitrag Max Muster', 'price' => 4466]);
-        $this->assertDatabaseHas('invoice_positions', ['description' => 'beitrag Jane Muster', 'price' => 4466]);
+        $this->assertDatabaseHas('invoice_positions', ['description' => 'beitrag Max Muster']);
+        $this->assertDatabaseHas('invoice_positions', ['description' => 'beitrag Jane Muster']);
     }
 
     public function testItSeparatesBillKinds(): void
@@ -105,21 +105,4 @@ class MassStoreActionTest extends TestCase
         $this->assertDatabaseCount('invoices', 2);
         $this->assertDatabaseCount('invoice_positions', 2);
     }
-
-    public function testItSeparatesSubscriptions(): void
-    {
-        $member1 = Member::factory()->defaults()->emailBillKind()
-            ->for(Subscription::factory()->forFee()->children([new Child('beitrag1 {name}', 4466)]))
-            ->create(['firstname' => 'Member1', 'lastname' => 'ln']);
-        $member2 = Member::factory()->defaults()->sameFamilyAs($member1)->emailBillKind()
-            ->for(Subscription::factory()->forFee()->children([new Child('beitrag2 {name}', 4467)]))
-            ->create(['firstname' => 'Member2']);
-
-        $this->postJson(route('invoice.mass-store'), ['year' => now()->addYear()->year])->assertOk();
-        $invoice = Invoice::first();
-
-        $this->assertDatabaseCount('invoice_positions', 2);
-        $this->assertDatabaseHas('invoice_positions', ['invoice_id' => $invoice->id, 'member_id' => $member1->id, 'description' => 'beitrag1 Member1 ln', 'price' => 4466]);
-        $this->assertDatabaseHas('invoice_positions', ['invoice_id' => $invoice->id, 'member_id' => $member2->id, 'description' => 'beitrag2 Member2 ln', 'price' => 4467]);
-    }
 }

tests/Feature/Member/PullMemberActionTest.php

@@ -7,7 +7,6 @@ use App\Country;
 use App\Fee;
 use App\Gender;
 use App\Group;
-use App\Member\Member;
 use App\Nationality;
 use App\Payment\Subscription;
 use App\Region;
@@ -143,20 +142,3 @@ it('testItPullsMemberWithNoSubscription', function () {
         'subscription_id' => null,
     ]);
 });
-
-it('doesnt set first subscription if fee matches', function () {
-    $this->loginNami();
-    Subscription::factory()->forFee(55)->create();
-    $otherSubscription = Subscription::factory()->forFee(55)->create();
-    $member = Member::factory()->defaults()->inNami(1001)->create(['subscription_id' => $otherSubscription->id]);
-    app(MemberFake::class)->shows(1000, 1001, [
-        'beitragsartId' => 55,
-    ]);
-
-    app(PullMemberAction::class)->handle(1000, 1001);
-
-    $this->assertDatabaseHas('members', [
-        'subscription_id' => $otherSubscription->id,
-        'id' => $member->id,
-    ]);
-});

tests/Feature/Member/ShowTest.php

@@ -149,7 +149,7 @@ class ShowTest extends TestCase
             ->for(Group::factory())
             ->for(Nationality::factory()->name('deutsch'))
             ->for(Subscription::factory()->forFee())
-            ->create(['firstname' => 'Max', 'lastname' => 'Muster', 'has_vk' => false, 'has_svk' => false]);
+            ->create(['firstname' => 'Max', 'lastname' => 'Muster']);
 
         $response = $this->get("/member/{$member->id}");