Merge pull request #11856 from firefly-iii/release-1772514544

🤖 Automatically merge the PR into the develop branch.

app/Helpers/Collector/GroupCollector.php

@@ -778,9 +778,10 @@ class GroupCollector implements GroupCollectorInterface
     {
         $newArray = $newJournal->toArray();
         if (array_key_exists('attachment_id', $newArray)) {
-            $attachmentId                                  = (int) $newJournal['attachment_id'];
-
-            $existingJournal['attachments'][$attachmentId] = ['id' => $attachmentId];
+            $attachmentId = (int) $newJournal['attachment_id'];
+            if (0 !== $attachmentId) {
+                $existingJournal['attachments'][$attachmentId] = ['id' => $attachmentId];
+            }
         }
 
         return $existingJournal;

app/Http/Controllers/Chart/ReportController.php

@@ -151,9 +151,8 @@ class ReportController extends Controller
         $cache->addProperty($end);
         $cache->addProperty($this->convertToPrimary);
         if ($cache->has()) {
-            // return response()->json($cache->get());
+            //             return response()->json($cache->get());
         }
-
         Log::debug('Going to do operations for accounts ', $accounts->pluck('id')->toArray());
         Log::debug(sprintf('Period: %s to %s', $start->toW3cString(), $end->toW3cString()));
         $format         = Navigation::preferredCarbonFormat($start, $end);
@@ -247,10 +246,6 @@ class ReportController extends Controller
             if ('1Y' === $preferredRange) {
                 $currentEnd = Navigation::endOfPeriod($currentEnd, $preferredRange);
             }
-            // 2026-03-01 similar fix for monthly ranges.
-            if ('1M' === $preferredRange) {
-                $currentEnd = Navigation::endOfPeriod($currentEnd, $preferredRange);
-            }
             Log::debug(sprintf('Start of sub-loop, current end is %s', $currentEnd->toW3cString()));
             while ($currentStart <= $currentEnd) {
                 Log::debug(sprintf('Current start: %s', $currentStart->toW3cString()));

app/Http/Controllers/TagController.php

@@ -246,12 +246,17 @@ class TagController extends Controller
         /** @var GroupCollectorInterface $collector */
         $collector    = app(GroupCollectorInterface::class);
 
+        // collect transaction journal IDs in repository,
+        // this makes the collector faster and more accurate.
+        $journalIds   = $this->repository->getJournalIds($tag);
+
         $collector
             ->setRange($start, $end)
             ->setLimit($pageSize)
             ->setPage($page)
+            ->setJournalIds($journalIds)
             ->withAccountInformation()
-            ->setTag($tag)
+            // ->setTag($tag)
             ->withBudgetInformation()
             ->withCategoryInformation()
             ->withAttachmentInformation()
@@ -296,6 +301,10 @@ class TagController extends Controller
         $path         = route('tags.show', [$tag->id, 'all']);
         $location     = $this->repository->getLocation($tag);
 
+        // collect transaction journal IDs in repository,
+        // this makes the collector faster and more accurate.
+        $journalIds   = $this->repository->getJournalIds($tag);
+
         /** @var GroupCollectorInterface $collector */
         $collector    = app(GroupCollectorInterface::class);
         $collector
@@ -303,7 +312,7 @@ class TagController extends Controller
             ->setLimit($pageSize)
             ->setPage($page)
             ->withAccountInformation()
-            ->setTag($tag)
+            ->setJournalIds($journalIds)
             ->withBudgetInformation()
             ->withCategoryInformation()
             ->withAttachmentInformation()

app/Listeners/Model/Account/UpdatesAccountInformation.php

@@ -51,7 +51,7 @@ class UpdatesAccountInformation implements ShouldQueue
 
     private function correctRuleActions(Account $account, array $oldData, Rule $rule): void
     {
-        $fields = ['set_source_account', 'set_destination_account'];
+        $fields = ['set_source_account', 'set_destination_account', 'convert_withdrawal', 'convert_deposit', 'convert_transfer'];
 
         Log::debug(sprintf('Check if rule #%d actions reference account #%d "%s"', $rule->id, $account->id, $account->name));
         $fixed  = 0;

app/Repositories/Tag/TagRepository.php

@@ -135,6 +135,12 @@ class TagRepository implements TagRepositoryInterface, UserGroupInterface
         });
     }
 
+    #[Override]
+    public function getJournalIds(Tag $tag): array
+    {
+        return $tag->transactionJournals->pluck('id')->toArray();
+    }
+
     public function getLocation(Tag $tag): ?Location
     {
         /** @var null|Location */

app/Repositories/Tag/TagRepositoryInterface.php

@@ -71,6 +71,8 @@ interface TagRepositoryInterface
 
     public function getAttachments(Tag $tag): Collection;
 
+    public function getJournalIds(Tag $tag): array;
+
     /**
      * Return location, or NULL.
      */

config/firefly.php

@@ -78,8 +78,8 @@ return [
         'running_balance_column' => (bool)envNonEmpty('USE_RUNNING_BALANCE', true), // this is only the default value, is not used.
         // see cer.php for exchange rates feature flag.
     ],
-    'version'                              => 'develop/2026-03-01',
-    'build_time'                           => 1772348761,
+    'version'                              => 'develop/2026-03-03',
+    'build_time'                           => 1772514431,
     'api_version'                          => '2.1.0', // field is no longer used.
     'db_version'                           => 28, // field is no longer used.
 

package-lock.json

@@ -9005,9 +9005,9 @@
             }
         },
         "node_modules/postcss": {
-            "version": "8.5.6",
-            "resolved": "https://registry.npmjs.org/postcss/-/postcss-8.5.6.tgz",
-            "integrity": "sha512-3Ybi1tAuwAP9s0r1UQ2J4n5Y0G05bJkpUIO0/bI9MhwmD70S5aTWbXGBwxHrelT+XM1k6dM0pk+SwNkpTRN7Pg==",
+            "version": "8.5.8",
+            "resolved": "https://registry.npmjs.org/postcss/-/postcss-8.5.8.tgz",
+            "integrity": "sha512-OW/rX8O/jXnm82Ey1k44pObPtdblfiuWnrd8X7GJ7emImCOstunGbXUpp7HdBrFQX6rJzn3sPT397Wp5aCwCHg==",
             "dev": true,
             "funding": [
                 {

routes/api.php

@@ -724,7 +724,6 @@ Route::group(
         'namespace'  => 'FireflyIII\Api\V1\Controllers\System',
         'prefix'     => 'v1/configuration',
         'as'         => 'api.v1.configuration.',
-        'middleware' => ['api-admin'],
     ],
     static function (): void {
         Route::get('', ['uses' => 'ConfigurationController@index', 'as' => 'index']);