chore(deps): update actions/setup-node action to v7 (stable1.0)

.github/workflows/appstore-build-publish.yml

@@ -71,7 +71,7 @@ jobs:
       - name: Set up node ${{ steps.versions.outputs.nodeVersion }}
         # Skip if no package.json
         if: ${{ steps.versions.outputs.nodeVersion }}
-        uses: actions/setup-node@249970729cb0ef3589644e2896645e5dc5ba9c38 # v6.5.0
+        uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
         with:
           node-version: ${{ steps.versions.outputs.nodeVersion }}
 

.github/workflows/cypress-component.yml

@@ -27,7 +27,7 @@ jobs:
         uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7
 
       - name: Set up node from version file
-        uses: actions/setup-node@249970729cb0ef3589644e2896645e5dc5ba9c38 # v6
+        uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7
         with:
           cache: 'npm'
           cache-dependency-path: package-lock.json

.github/workflows/cypress-custom.yml

@@ -88,7 +88,7 @@ jobs:
           path: apps/${{ env.APP_NAME }}
 
       - name: Set up node from version file
-        uses: actions/setup-node@249970729cb0ef3589644e2896645e5dc5ba9c38 # v6
+        uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7
         with:
           cache: 'npm'
           cache-dependency-path: apps/${{ env.APP_NAME}}/package-lock.json
@@ -103,7 +103,7 @@ jobs:
 
 
       - name: Use Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@249970729cb0ef3589644e2896645e5dc5ba9c38 # v6
+        uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7
         with:
           node-version: ${{ matrix.node-version }}
 

.github/workflows/lint-eslint.yml

@@ -68,7 +68,7 @@ jobs:
           fallbackNpm: '^10'
 
       - name: Set up node ${{ steps.versions.outputs.nodeVersion }}
-        uses: actions/setup-node@249970729cb0ef3589644e2896645e5dc5ba9c38 # v6.5.0
+        uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
         with:
           node-version: ${{ steps.versions.outputs.nodeVersion }}
 

.github/workflows/lint-stylelint.yml

@@ -37,7 +37,7 @@ jobs:
           fallbackNpm: '^10'
 
       - name: Set up node ${{ steps.versions.outputs.nodeVersion }}
-        uses: actions/setup-node@249970729cb0ef3589644e2896645e5dc5ba9c38 # v6.5.0
+        uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
         with:
           node-version: ${{ steps.versions.outputs.nodeVersion }}
 

.github/workflows/node.yml

@@ -65,7 +65,7 @@ jobs:
           fallbackNpm: '^10'
 
       - name: Set up node ${{ steps.versions.outputs.nodeVersion }}
-        uses: actions/setup-node@249970729cb0ef3589644e2896645e5dc5ba9c38 # v6.5.0
+        uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
         with:
           node-version: ${{ steps.versions.outputs.nodeVersion }}
 

.github/workflows/npm-audit-fix.yml

@@ -46,7 +46,7 @@ jobs:
           fallbackNpm: '^10'
 
       - name: Set up node ${{ steps.versions.outputs.nodeVersion }}
-        uses: actions/setup-node@249970729cb0ef3589644e2896645e5dc5ba9c38 # v6.5.0
+        uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
         with:
           node-version: ${{ steps.versions.outputs.nodeVersion }}
 

.github/workflows/openapi.yml

@@ -62,7 +62,7 @@ jobs:
 
       - name: Set up node ${{ steps.node_versions.outputs.nodeVersion }}
         if: ${{ steps.node_versions.outputs.nodeVersion }}
-        uses: actions/setup-node@249970729cb0ef3589644e2896645e5dc5ba9c38 # v6.5.0
+        uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
         with:
           node-version: ${{ steps.node_versions.outputs.nodeVersion }}
 

lib/Activity/ActivityManager.php

@@ -45,7 +45,14 @@ public function __construct(
 	) {
 	}
 
-	public function triggerEvent($objectType, $object, $subject, $additionalParams = [], $author = null) {
+	/**
+	 * @param array|null|string $additionalParams
+	 * @param (array|null)[]|null|string $author
+	 *
+	 * @psalm-param 'tables_row'|'tables_table' $objectType
+	 * @psalm-param array{before: array|null, after: array|null}|null|string $author
+	 */
+	public function triggerEvent(string $objectType, Row2|Table $object, string $subject, array|string|null $additionalParams = [], array|string|null $author = null) {
 		if ($author === null) {
 			$author = $this->userId;
 		}
@@ -61,7 +68,11 @@ public function triggerEvent($objectType, $object, $subject, $additionalParams =
 		}
 	}
 
-	public function triggerUpdateEvents($objectType, ChangeSet $changeSet, $subject) {
+	/**
+	 * @psalm-param 'tables_table' $objectType
+	 * @psalm-param 'table_update' $subject
+	 */
+	public function triggerUpdateEvents(string $objectType, ChangeSet $changeSet, string $subject) {
 		$previousEntity = $changeSet->getBefore();
 		$entity = $changeSet->getAfter();
 		$events = [];
@@ -98,7 +109,10 @@ public function triggerUpdateEvents($objectType, ChangeSet $changeSet, $subject)
 		}
 	}
 
-	private function createEvent($objectType, $object, $subject, $additionalParams = [], $author = null) {
+	/**
+	 * @psalm-param array{before?: mixed, after?: mixed} $additionalParams
+	 */
+	private function createEvent($objectType, $object, string $subject, array $additionalParams = [], $author = null) {
 		if ($object instanceof Table) {
 			$objectTitle = $object->getTitle();
 			$table = $object;
@@ -200,7 +214,7 @@ private function sendToUsers(IEvent $event, $object) {
 		}
 	}
 
-	public function getActivityFormat($language, $subjectIdentifier, $subjectParams = [], $ownActivity = false) {
+	public function getActivityFormat(string $language, $subjectIdentifier, array $subjectParams = [], bool $ownActivity = false) {
 		$subject = '';
 		$l = $this->l10nFactory->get(Application::APP_ID, $language);
 

lib/Activity/ChangeSet.php

@@ -24,11 +24,11 @@ public function __construct(
 		}
 	}
 
-	public function setBefore($before) {
+	public function setBefore(Entity $before) {
 		$this->before = clone $before;
 	}
 
-	public function setAfter($after) {
+	public function setAfter(Entity $after) {
 		$this->after = clone $after;
 	}
 

lib/Db/LegacyRowMapper.php

@@ -82,7 +82,7 @@ public function find(int $id): LegacyRow {
 		return $this->findEntity($qb);
 	}
 
-	private function buildFilterByColumnType($qb, array $filter, string $filterId): ?IQueryFunction {
+	private function buildFilterByColumnType(IQueryBuilder $qb, array $filter, string $filterId): ?IQueryFunction {
 		try {
 			$columnQbClassName = 'OCA\Tables\Db\ColumnTypes\\';
 			$type = explode('-', $filter['columnType'])[0];

lib/Db/RowCellMapperSuper.php

@@ -41,7 +41,10 @@ public function formatEntity(Column $column, RowCellSuper $cell) {
 	 * Transform value from a filter rule to the actual query parameter used
 	 * for constructing the view filter query
 	 */
-	public function filterValueToQueryParam(Column $column, mixed $value): mixed {
+	/**
+	 * @param array|float|null|string $value
+	 */
+	public function filterValueToQueryParam(Column $column, array|string|float|null $value): mixed {
 		return $value;
 	}
 

lib/Db/RowCellSuper.php

@@ -80,7 +80,7 @@
 		$this->setColumnId($columnId);
 	}
 
-	public function setValueWrapper($value) {
+	public function setValueWrapper(array|float|null $value) {
 		$this->setValue($value);
 	}
 }

lib/Model/RowDataInput.php

@@ -24,7 +24,10 @@ class RowDataInput implements ArrayAccess, Iterator {
 	/** @psalm-var array<array{'columnId': int, 'value': mixed}> */
 	protected array $data = [];
 
-	public function add(int $columnId, mixed $value): self {
+	/**
+	 * @param array|float|int|null|string $value
+	 */
+	public function add(int $columnId, array|string|int|float|null $value): self {
 		$this->data[] = [self::DATA_KEY => $columnId, self::DATA_VAL => $value];
 		return $this;
 	}

lib/Service/ImportService.php

@@ -501,18 +501,21 @@
 		}
 	}
 
-	private function valueToDateTimeImmutable(mixed $value): ?DateTimeImmutable {
+	/**
+	 * @param null|string $value
+	 */
+	private function valueToDateTimeImmutable(string|null $value): ?DateTimeImmutable {
 		if (
 			$value === false
 			|| $value === null
 			|| (is_string($value)
 				&& mb_strlen($value) < 3 // Let pass potential 3-letter month names
 				&& preg_match('/\d/', $value) !== 1) // or anything containing a digit
 		) {
 			return null;
 		}
 		try {
 			$dt = Date::excelToDateTimeObject($value);
 			Date::roundMicroseconds($dt);
 			return DateTimeImmutable::createFromMutable($dt);
 		} catch (TypeError) {

lib/Service/ValueObject/ViewColumnInformation.php

@@ -98,7 +98,7 @@ public function jsonSerialize(): array {
 		return $this->data;
 	}
 
-	protected function ensureType(string $offset, mixed $value): int|bool {
+	protected function ensureType(string $offset, bool|int $value): int|bool {
 		return match ($offset) {
 			self::KEY_ID,
 			self::KEY_ORDER => (int)$value,

vendor-bin/rector/composer.json

@@ -0,0 +1 @@
+{}