---
id: change/bcquality/157
type: change
title: "#157 18 more AL/BC patterns: data-modeling, testing, style, security, error-handling, ui, upgrade, web-services, appsource"
summary: BCQuality adds 18 new knowledge articles covering data modeling, testing, style, security, error handling, UI, upgrade, web services, and AppSource patterns with code examples and best practices.
tier: official
language: en
tags:
  - feature
  - main
system: development
review:
  state: reviewed
  by: opus
  at: "2026-10-08T02:03:52.830Z"
  flags: []
generated:
  at: "2026-10-08T01:26:53.141Z"
  pipeline: 0.2.0
  prompts:
    extract-change: 1
  input_hash: f63943dcfdfd1bb776441b2f9f08966f97295a01
evidence:
  - kind: code
    url: https://github.com/microsoft/BCQuality/pull/157
    title: "microsoft/BCQuality#157: 18 more AL/BC patterns: data-modeling, testing, style, security, error-handling, ui, upgrade, web-services, appsource"
    date: "2026-09-29"
    commit: f63943dcfdfd1bb776441b2f9f08966f97295a01
    t: null
    quote: 18 more knowledge articles generalized from patterns observed across real AppSource/PTE Business Central development
links:
  learn: []
  objects: []
  features: []
  topics: []
  localizations: []
  videos: []
  posts: []
  guidelines: []
  changes: []
number: 157
repo: microsoft/BCQuality
source_id: bcquality-prs
url: https://github.com/microsoft/BCQuality/pull/157
kind: pr
base_branch: main
major: null
merged_at: "2026-09-29T15:12:51Z"
author: MichaelDieringer
author_type: User
community_contribution: false
labels: []
change_class: code
change_kind: feature
behavior_change: false
breaking: false
files:
  count: 59
  additions: 1612
  deletions: 2
  al: 59
apps: []
objects_touched: []
objects_unjoined:
  - path: microsoft/knowledge/appsource/release-must-update-app-version.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/data-modeling/code-must-not-change-workdate.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/data-modeling/code-must-not-change-workdate.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/data-modeling/code-must-not-change-workdate.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/data-modeling/pictures-must-use-media-not-blob.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/data-modeling/pictures-must-use-media-not-blob.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/data-modeling/pictures-must-use-media-not-blob.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/data-modeling/table-design-must-match-bc-table-type-conventions.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/data-modeling/table-design-must-match-bc-table-type-conventions.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/data-modeling/table-design-must-match-bc-table-type-conventions.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/error-handling/defensive-vs-offensive-code-must-match-blast-radius.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/error-handling/defensive-vs-offensive-code-must-match-blast-radius.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/error-handling/defensive-vs-offensive-code-must-match-blast-radius.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/error-handling/log-writes-must-survive-rollback.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/error-handling/log-writes-must-survive-rollback.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/error-handling/log-writes-must-survive-rollback.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/security/exposed-objects-must-be-in-a-permission-set.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/security/exposed-objects-must-be-in-a-permission-set.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/security/exposed-objects-must-be-in-a-permission-set.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/style/al-comments-must-not-restate-what-code-already-shows.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/style/al-comments-must-not-restate-what-code-already-shows.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/style/al-comments-must-not-restate-what-code-already-shows.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/style/pages-must-not-contain-business-logic.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/style/pages-must-not-contain-business-logic.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/style/pages-must-not-contain-business-logic.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/style/source-organized-by-feature-not-object-type.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/bcpt-scenarios-must-be-app-specific.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/bcpt-scenarios-must-be-app-specific.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/bcpt-scenarios-must-be-app-specific.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/given-blocks-must-cover-full-precondition-chain.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/given-blocks-must-cover-full-precondition-chain.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/given-blocks-must-cover-full-precondition-chain.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/test-feature-scenario-tags.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/test-feature-scenario-tags.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/test-feature-scenario-tags.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/test-one-when-per-test.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/test-one-when-per-test.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/test-one-when-per-test.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/ui-test-codeunit-naming.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/ui-test-codeunit-naming.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/testing/ui-test-codeunit-naming.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/ui/page-design-must-match-bc-page-type-conventions.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/ui/page-design-must-match-bc-page-type-conventions.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/ui/page-design-must-match-bc-page-type-conventions.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/upgrade/upgrade-tag-logic-must-not-nest-deeply.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/upgrade/upgrade-tag-logic-must-not-nest-deeply.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/upgrade/upgrade-tag-logic-must-not-nest-deeply.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/web-services/api-page-least-privilege-write-access.bad.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/web-services/api-page-least-privilege-write-access.good.al
    status: added
    reason: not-in-snapshot
  - path: microsoft/knowledge/web-services/api-page-least-privilege-write-access.md
    status: added
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-appsource-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-data-modeling-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-error-handling-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-security-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-style-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-testing-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-ui-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-upgrade-review.md
    status: modified
    reason: not-in-snapshot
  - path: microsoft/skills/review/al-web-services-review.md
    status: modified
    reason: not-in-snapshot
obsoletions: []
backports: []
fixes_issues: []
work_items: []
systems:
  - development
quote: 18 more knowledge articles generalized from patterns observed across real AppSource/PTE Business Central development
joined_against:
  major: null
  commit: null
---

# #157 18 more AL/BC patterns: data-modeling, testing, style, security, error-handling, ui, upgrade, web-services, appsource

> BCQuality adds 18 new knowledge articles covering data modeling, testing, style, security, error handling, UI, upgrade, web services, and AppSource patterns with code examples and best practices.

[Pull request](https://github.com/microsoft/BCQuality/pull/157) · merged into `main` on 2026-09-29 by MichaelDieringer · 59 files (+1612 -2), 59 AL · feature · tier official · reviewed (checked by Opus)

## What changed

- 18 articles across data modeling (WorkDate, Media for pictures, table-type conventions), testing (BCPT, tagging, test structure) and style
- Security, error handling, UI, upgrade, web services and AppSource guidance on permission sets, rollback-safe logging, page types, upgrade tag nesting, API write access and version bumps
- Sibling good and bad AL samples are included where a code contrast helps
- Four drafted articles were dropped as duplicates or overlaps of existing files
- Review skills updated to reference the new articles

## AL objects touched

- `microsoft/knowledge/appsource/release-must-update-app-version.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/data-modeling/code-must-not-change-workdate.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/data-modeling/code-must-not-change-workdate.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/data-modeling/code-must-not-change-workdate.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/data-modeling/pictures-must-use-media-not-blob.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/data-modeling/pictures-must-use-media-not-blob.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/data-modeling/pictures-must-use-media-not-blob.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/data-modeling/table-design-must-match-bc-table-type-conventions.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/data-modeling/table-design-must-match-bc-table-type-conventions.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/data-modeling/table-design-must-match-bc-table-type-conventions.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/error-handling/defensive-vs-offensive-code-must-match-blast-radius.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/error-handling/defensive-vs-offensive-code-must-match-blast-radius.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/error-handling/defensive-vs-offensive-code-must-match-blast-radius.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/error-handling/log-writes-must-survive-rollback.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/error-handling/log-writes-must-survive-rollback.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/error-handling/log-writes-must-survive-rollback.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/security/exposed-objects-must-be-in-a-permission-set.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/security/exposed-objects-must-be-in-a-permission-set.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/security/exposed-objects-must-be-in-a-permission-set.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/style/al-comments-must-not-restate-what-code-already-shows.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/style/al-comments-must-not-restate-what-code-already-shows.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/style/al-comments-must-not-restate-what-code-already-shows.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/style/pages-must-not-contain-business-logic.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/style/pages-must-not-contain-business-logic.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/style/pages-must-not-contain-business-logic.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/style/source-organized-by-feature-not-object-type.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/bcpt-scenarios-must-be-app-specific.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/bcpt-scenarios-must-be-app-specific.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/bcpt-scenarios-must-be-app-specific.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/given-blocks-must-cover-full-precondition-chain.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/given-blocks-must-cover-full-precondition-chain.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/given-blocks-must-cover-full-precondition-chain.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/test-feature-scenario-tags.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/test-feature-scenario-tags.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/test-feature-scenario-tags.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/test-one-when-per-test.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/test-one-when-per-test.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/test-one-when-per-test.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/ui-test-codeunit-naming.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/ui-test-codeunit-naming.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/testing/ui-test-codeunit-naming.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/ui/page-design-must-match-bc-page-type-conventions.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/ui/page-design-must-match-bc-page-type-conventions.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/ui/page-design-must-match-bc-page-type-conventions.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/upgrade/upgrade-tag-logic-must-not-nest-deeply.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/upgrade/upgrade-tag-logic-must-not-nest-deeply.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/upgrade/upgrade-tag-logic-must-not-nest-deeply.md`: added, not in the BC? snapshot yet
- `microsoft/knowledge/web-services/api-page-least-privilege-write-access.bad.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/web-services/api-page-least-privilege-write-access.good.al`: added, not in the BC? snapshot yet
- `microsoft/knowledge/web-services/api-page-least-privilege-write-access.md`: added, not in the BC? snapshot yet
- `microsoft/skills/review/al-appsource-review.md`: modified, not in the BC? snapshot
- `microsoft/skills/review/al-data-modeling-review.md`: modified, not in the BC? snapshot
- `microsoft/skills/review/al-error-handling-review.md`: modified, not in the BC? snapshot
- `microsoft/skills/review/al-security-review.md`: modified, not in the BC? snapshot
- `microsoft/skills/review/al-style-review.md`: modified, not in the BC? snapshot
- `microsoft/skills/review/al-testing-review.md`: modified, not in the BC? snapshot
- `microsoft/skills/review/al-ui-review.md`: modified, not in the BC? snapshot
- `microsoft/skills/review/al-upgrade-review.md`: modified, not in the BC? snapshot
- `microsoft/skills/review/al-web-services-review.md`: modified, not in the BC? snapshot

## Context

- Systems: development
- From the description: "18 more knowledge articles generalized from patterns observed across real AppSource/PTE Business Central development"

Source: metadata of a merged pull request in microsoft/BCQuality (bcquality): title, labels, changed paths and counts, with a summary in our words. No source text and no description beyond one short quote (D10, D61).