Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
80 changes: 17 additions & 63 deletions .github/workflows/code-quality.yml
Original file line number Diff line number Diff line change
@@ -1,70 +1,24 @@
name: Code Quality

on:
push:
branches: [main, development, feature/**, bugfix/**, hotfix/**]
pull_request:
branches: [main, master, development]

concurrency:
group: quality-${{ github.head_ref || github.ref }}
cancel-in-progress: true

jobs:
php-checks:
name: ${{ matrix.check.name }}
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
check:
- { name: "PHP Lint", command: "composer lint" }
- { name: "PHPCS", command: "./vendor/bin/phpcs --standard=phpcs.xml" }
- { name: "PHPMD", command: "./vendor/bin/phpmd lib text phpmd.xml" }
- { name: "Psalm", command: "./vendor/bin/psalm --threads=1 --no-cache --output-format=github" }
- { name: "PHPStan", command: "./vendor/bin/phpstan analyse --memory-limit=1G" }
- { name: "PHPUnit", command: "./vendor/bin/phpunit --colors=always" }

steps:
- name: Checkout
uses: actions/checkout@v4

- name: Setup PHP
uses: shivammathur/setup-php@v2
with:
php-version: '8.1'
extensions: mbstring, xml, ctype, iconv, intl, dom, filter, gd, json, posix, zip, soap
tools: composer:v2

- name: Cache Composer dependencies
uses: actions/cache@v4
with:
path: vendor
key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
restore-keys: ${{ runner.os }}-composer-

- name: Install dependencies
run: composer install --no-progress --prefer-dist --optimize-autoloader

- name: ${{ matrix.check.name }}
run: ${{ matrix.check.command }}
frontend-quality:
name: Frontend Quality
runs-on: ubuntu-latest

steps:
- name: Checkout
uses: actions/checkout@v4

- name: Setup Node.js
uses: actions/setup-node@v4
with:
node-version: '20'
cache: 'npm'

- name: Install dependencies
run: npm ci

- name: ESLint
run: npm run lint

- name: Stylelint
run: npm run stylelint
quality:
uses: ConductionNL/.github/.github/workflows/quality.yml@main
with:
app-name: softwarecatalog
php-version: "8.3"
nextcloud-ref: stable32
enable-psalm: true
enable-phpstan: true
enable-phpmetrics: true
enable-frontend: true
enable-phpcs: false
enable-eslint: true
enable-phpunit: true
enable-newman: false
additional-apps: '[{"repo":"ConductionNL/openregister","app":"openregister"}]'
2 changes: 1 addition & 1 deletion appinfo/info.xml
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,7 @@ Free and open source under the EUPL license.

Vrij en open source onder de EUPL-licentie.
]]></description>
<version>0.1.140</version>
<version>0.1.141</version>
<licence>agpl</licence>
<author mail="info@conduction.nl" homepage="https://www.conduction.nl/">Conduction</author>
<namespace>SoftwareCatalog</namespace>
Expand Down
37 changes: 37 additions & 0 deletions issues/457.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
# #457 — Koppeling: verwijderen geeft een 400-error

**Status:** OPEN | **Labels:** Aanbod, IGS nieuw
**Auteur:** @Makkmetp | **Datum:** 2026-03-05
**Link:** https://github.com/VNG-Realisatie/Softwarecatalogus/issues/457

---

## Beschrijving

Het verwijderen van een koppeling geeft een 400-error. Zowel bij een geimporteerde als nieuw aangemaakte leverancier.

Ga via Dashboard naar je koppelingen overzicht.
Klik bij een koppeling op ... Acties op Bekijken.
Verwijder via ...Acties > Verwijderen de koppeling

<img width="1161" height="784" alt="Image" src="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/user-attachments/assets/79ff1634-3267-4ac2-87cd-5ec29f2ad664" />

<img width="1211" height="596" alt="Image" src="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/user-attachments/assets/d4cedc14-051a-4cae-bbd9-d2bbdfeed63d" />

Dit is getest met meerdere koppelingen. Ook met koppelingen waar nog geen gebruik van wordt gemaakt door een gemeente.
Graag ervoor zorgen dat koppelingen volgens integriteitsregels verwijderd kunnen worden en geen 400-error geven.

---

## Reacties (1)

### Reactie 1 — @github-actions (2026-03-05)

Bedankt voor het aanmaken van deze issue! 👋

Voor meer informatie over onze spelregels voor het schrijven van issues, bekijk alsjeblieft onze [issues.md](../../issues/issues.md) documentatie.

Belangrijke punten om te controleren:
- ✍️ Is de beschrijving helder en compleet?
- 📋 Zijn er duidelijke acceptatiecriteria toegevoegd?
- 🎯 Is de context voldoende beschreven?
32 changes: 25 additions & 7 deletions phpunit.xml
Original file line number Diff line number Diff line change
@@ -1,20 +1,38 @@
<?xml version="1.0" encoding="UTF-8"?>
<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd"
xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd"
bootstrap="tests/bootstrap.php"
colors="true">
colors="true"
cacheDirectory=".phpunit.cache"
executionOrder="depends,defects"
requireCoverageMetadata="false"
beStrictAboutCoverageMetadata="true"
beStrictAboutOutputDuringTests="true"
failOnRisky="true"
failOnWarning="false"
failOnDeprecation="false">

<testsuites>
<testsuite name="Unit Tests">
<directory>tests/unit</directory>
</testsuite>
<testsuite name="Integration Tests">
<directory>tests/Integration</directory>
<directory>tests/Unit</directory>
</testsuite>
</testsuites>

<coverage processUncoveredFiles="true">
<source>
<include>
<directory suffix=".php">lib/</directory>
</include>
</source>

<coverage>
<report>
<clover outputFile="coverage/clover.xml"/>
<html outputDirectory="coverage/html"/>
<text outputFile="php://stdout" showUncoveredFiles="false"/>
</report>
</coverage>

<php>
<env name="PHPUNIT_RUN" value="1"/>
</php>
</phpunit>
Loading
Loading