release the beta code

.github/workflows/beta-release.yaml

@@ -4,7 +4,6 @@ on:
   push:
     branches:
       - feature/newopenregister
-      # - beta
 
 jobs:
   release-management:
@@ -31,7 +30,7 @@ jobs:
           git fetch origin main
           main_version=$(git show origin/main:appinfo/info.xml | grep -oP '(?<=<version>)[^<]+' || echo "")
 
-          # Get current version from feature/newopenregister branch
+          # Get current version from current branch
           current_version=$(grep -oP '(?<=<version>)[^<]+' appinfo/info.xml || echo "")
 
           # Split main version into parts
@@ -43,7 +42,7 @@ jobs:
           # Extract beta counter from current version if it exists
           beta_counter=1
           if [[ $current_version =~ -beta\.([0-9]+)$ ]]; then
-            # If current patch version is still ahead of master, increment counter
+            # If current patch version is still ahead of main, increment counter
             current_patch=$(echo $current_version | grep -oP '^[0-9]+\.[0-9]+\.(\d+)' | cut -d. -f3)
             if [ "$current_patch" -eq "$next_patch" ]; then
               beta_counter=$((BASH_REMATCH[1] + 1))
@@ -92,7 +91,7 @@ jobs:
 
       # Stap 9: Voer npm install, build en composer install uit
       - run: npm ci
-      - run: npm run build
+      - run: npm run dev
       - run: composer install --no-dev
 
       # Stap 10: Kopieer de bestanden naar de package directory
@@ -152,7 +151,7 @@ jobs:
         id: version
         uses: codacy/git-version@2.7.1
         with:
-          release-branch: feature/newopenregister
+          release-branch: beta-release
 
       # Stap 14: Extraheer repository description (optioneel)
       - name: Extract repository description
@@ -174,6 +173,7 @@ jobs:
           name: Beta Release ${{ env.NEW_VERSION }}
           draft: false
           prerelease: true
+          skipIfReleaseExists: true
 
       # Stap 18: Voeg het tarball toe als asset aan de GitHub release
       - name: Attach tarball to GitHub release

appinfo/info.xml

@@ -22,7 +22,7 @@ Submit a [bug report](https://github.com/OpenCatalogi/.github/issues/new/choose)
 
 Submit a [feature request](https://github.com/OpenCatalogi/.github/issues/new/choose).
     ]]></description>
-    <version>0.1.137-beta.2</version>
+    <version>0.1.137-beta.28</version>
     <licence>agpl</licence>
     <category>organization</category>
     <author mail="info@conduction.nl" homepage="https://www.conduction.nl/">Conduction</author>
@@ -39,6 +39,12 @@ Submit a [feature request](https://github.com/OpenCatalogi/.github/issues/new/ch
         <database min-version="8.0">mysql</database>
     </dependencies>
 
+	<repair-steps>
+		<post-migration>
+			<step>OCA\SoftwareCatalog\Repair\InitializeSettings</step>
+		</post-migration>
+	</repair-steps>
+
 	<background-jobs>
 		<job>OCA\SoftwareCatalog\Cron\OrganizationSync</job>
 	</background-jobs>

appinfo/routes.php

@@ -132,6 +132,9 @@
         ['name' => 'settings#updateSyncConfig', 'url' => '/api/settings/sync/config', 'verb' => 'POST'],
         ['name' => 'settings#syncOrganisations', 'url' => '/api/settings/sync/organisations', 'verb' => 'POST'],
 
+        // User Profile endpoint
+        ['name' => 'contactpersonen#getMe', 'url' => '/api/me', 'verb' => 'GET'],
+
         // Contactpersonen Management endpoints
         ['name' => 'contactpersonen#getContactpersonen', 'url' => '/api/contactpersonen/organisation/{organisationId}', 'verb' => 'GET'],
         ['name' => 'contactpersonen#getContactPersonsWithUserDetailsForOrganization', 'url' => '/api/contactpersonen/organisation/{organizationUuid}/with-user-details', 'verb' => 'GET'],

lib/AppInfo/Application.php

@@ -137,7 +137,7 @@ public function register(IRegistrationContext $context): void
 
         // Register TEST event listener for easily triggerable Nextcloud events
         $context->registerEventListener(UserLoggedInEvent::class, TestEventListener::class);
-        
+
         // Register event listeners for OpenRegister events
         $context->registerEventListener(ObjectCreatedEvent::class, SoftwareCatalogEventListener::class);
         $context->registerEventListener(ObjectUpdatedEvent::class, SoftwareCatalogEventListener::class);
@@ -298,6 +298,23 @@ public function register(IRegistrationContext $context): void
                 $container->get('Psr\Log\LoggerInterface')
             );
         });
+
+        // Register ContactpersonenController with explicit dependencies for /me endpoint
+        $context->registerService(\OCA\SoftwareCatalog\Controller\ContactpersonenController::class, function ($container) {
+            return new \OCA\SoftwareCatalog\Controller\ContactpersonenController(
+                self::APP_ID,
+                $container->get('OCP\IRequest'),
+                $container->get(SettingsService::class),
+                $container->get('OCA\SoftwareCatalog\Service\SoftwareCatalogue\ContactPersonHandler'),
+                $container->get(\OCA\SoftwareCatalog\Service\ContactpersoonService::class),
+                $container->get('OCP\IUserManager'),
+                $container->get('OCP\IGroupManager'),
+                $container->get('OCP\IUserSession'),
+                $container,
+                $container->get('OCP\Security\ISecureRandom'),
+                $container->get('Psr\Log\LoggerInterface')
+            );
+        });
     }
 
     /**
@@ -309,135 +326,25 @@ public function register(IRegistrationContext $context): void
      */
     public function boot(IBootContext $context): void
     {
+        // Initialization is now handled by the Repair step (InitializeSettings)
+        // which runs only during app install/upgrade, not on every request.
+        // See lib/Repair/InitializeSettings.php
+
         $container = $context->getServerContainer();
         $logger = $container->get(LoggerInterface::class);
 
-        try {
-            $config = $container->get(IAppConfig::class);
-            $appManager = $container->get(IAppManager::class);
-            $currentAppVersion = $appManager->getAppVersion(self::APP_ID);
-            $lastInitializedVersion = $config->getValueString(self::APP_ID, 'last_initialized_version', '');
-
-            $logger->info('SoftwareCatalog boot: Version check', [
-                'currentVersion' => $currentAppVersion,
-                'lastInitializedVersion' => $lastInitializedVersion,
-                'versionChanged' => $lastInitializedVersion !== $currentAppVersion
-            ]);
-
-            // Skip initialization during cron/CLI context without a user.
-            // RBAC checks require an authenticated user to have proper permissions.
-            $userSession = $container->get(\OCP\IUserSession::class);
-            if ($userSession->getUser() === null) {
-                $logger->debug('SoftwareCatalog boot: Skipping in cron/CLI context (no user)');
-                return;
-            }
-
-            // Check if we actually have a valid configuration, not just version matching
-            $needsInitialization = false;
-            $initReason = '';
-
-            if ($lastInitializedVersion !== $currentAppVersion || empty($lastInitializedVersion)) {
-                $needsInitialization = true;
-                $initReason = empty($lastInitializedVersion) ? 'never_initialized' : 'version_changed';
-            } else {
-                // Even if version matches, check if we have valid configuration
-                $hasValidConfig = $config->getValueString(self::APP_ID, 'voorzieningen_organisatie_schema', '') !== '' ||
-                                  $config->getValueString(self::APP_ID, 'organization_schema', '') !== '';
-
-                if (!$hasValidConfig) {
-                    $needsInitialization = true;
-                    $initReason = 'missing_configuration';
-                    $logger->warning('SoftwareCatalog boot: No valid configuration found despite version match', [
-                        'currentVersion' => $currentAppVersion,
-                        'lastInitializedVersion' => $lastInitializedVersion
-                    ]);
-                }
-            }
-
-            if ($needsInitialization) {
-                $logger->info('SoftwareCatalog boot: Starting initialization', [
-                    'reason' => $initReason,
-                    'currentVersion' => $currentAppVersion,
-                    'lastInitializedVersion' => $lastInitializedVersion
-                ]);
-
-                try {
-                    $settingsService = $container->get(SettingsService::class);
-                    $initResult = $settingsService->initialize();
-
-                    $logger->info('SoftwareCatalog boot: Initialization completed', [
-                        'result' => $initResult,
-                        'hasErrors' => !empty($initResult['errors'])
-                    ]);
-
-                    // Only update version if initialization was actually successful
-                    if (empty($initResult['errors']) &&
-                        ($initResult['autoConfigured'] || $initResult['fullyConfigured'])) {
-                        $config->setValueString(self::APP_ID, 'last_initialized_version', $currentAppVersion);
-                        $logger->info('SoftwareCatalog boot: Version updated to ' . $currentAppVersion . ' (successful init)');
-                    } else {
-                        $logger->warning('SoftwareCatalog boot: Initialization incomplete, not updating version', [
-                            'errors' => $initResult['errors'] ?? [],
-                            'autoConfigured' => $initResult['autoConfigured'] ?? false,
-                            'fullyConfigured' => $initResult['fullyConfigured'] ?? false
-                        ]);
-                    }
-
-                } catch (\RuntimeException $e) {
-                    // Don't update version if OpenRegister is not available
-                    $logger->warning('SoftwareCatalog boot: OpenRegister not available during initialization', [
-                        'exception' => $e->getMessage(),
-                        'initReason' => $initReason
-                    ]);
-                } catch (\Exception $e) {
-                    $logger->error('SoftwareCatalog boot: Initialization failed', [
-                        'exception' => $e->getMessage(),
-                        'trace' => $e->getTraceAsString(),
-                        'initReason' => $initReason
-                    ]);
-                }
-            } else {
-                $logger->debug('SoftwareCatalog boot: Skipping initialization (version unchanged and config valid)');
-            }
-
-        } catch (\Exception $e) {
-            // Log error but don't fail the boot process
-            $logger->error('SoftwareCatalog boot error during version check: ' . $e->getMessage(), [
-                'exception' => $e,
-                'trace' => $e->getTraceAsString()
-            ]);
-        }
-
         // Register background job for organization contact synchronization
         try {
             $jobList = $container->get('OCP\BackgroundJob\IJobList');
             if (!$jobList->has(\OCA\SoftwareCatalog\BackgroundJob\OrganizationContactSyncJob::class, null)) {
                 $jobList->add(\OCA\SoftwareCatalog\BackgroundJob\OrganizationContactSyncJob::class);
-                $logger->info('SoftwareCatalog boot: Background job registered');
+                $logger->debug('SoftwareCatalog boot: Background job registered');
             }
         } catch (\Exception $e) {
             $logger->error('SoftwareCatalog boot: Failed to register background job', [
                 'exception' => $e->getMessage()
             ]);
         }
-
-        // Check if initial sync has been done
-        try {
-            $config = $container->get(IAppConfig::class);
-            $initialSyncDone = $config->getValueString(self::APP_ID, 'initial_sync_done', 'false');
-            if ($initialSyncDone === 'false') {
-                // Mark as done to prevent repeated attempts
-                $config->setValueString(self::APP_ID, 'initial_sync_done', 'true');
-                $logger->info('SoftwareCatalog boot: Initial sync flag set');
-            }
-        } catch (\Exception $e) {
-            // Log but don't fail
-            $logger->error('SoftwareCatalog boot error during sync check: ' . $e->getMessage(), [
-                'exception' => $e->getMessage()
-            ]);
-        }
-
-
     }