File as GitHub Issue repo: frappe/erpnext

Push this scan report to `frappe/erpnext`

Click the green button below to open GitHub’s new-issue form, pre-filled with the report title, summary table, top findings, and an embedded score-card image. No authentication needed — you review on GitHub before submitting. Repobility is credited as the scanner.

Embedded score card image

This image will render at the top of the issue body. Hosted on Repobility, refreshes automatically after re-scans.

Issue title

`self.enable_payment_schedule_in_print` used but never assigned in __init__

Curate findings to include

Pick exactly which findings appear in the issue body. By default the top 5 are included. Uncheck noise, check what matters.

Top 5 (default)

Severity	Rule	Title	File:line
CRIT	`MINED116`	[MINED116] Workflow uses `secrets.CODECOV_TOKEN` on a `pull_request` trigger: This workfl…	`.github/workflows/server-tests-mariadb.…:164`
HIGH	`MINED108`	[MINED108] `self.enable_accounting_dimensions` used but never assigned in __init__: Metho…	`erpnext/accounts/doctype/accounts_setti…:138`
HIGH	`MINED108`	[MINED108] `self.get` used but never assigned in __init__: Method `validate` of class `Ac…	`erpnext/accounts/doctype/accounts_setti…:128`
HIGH	`MINED108`	[MINED108] `self.get` used but never assigned in __init__: Method `validate` of class `Ac…	`erpnext/accounts/doctype/accounts_setti…:122`
HIGH	`MINED108`	[MINED108] `self.enable_payment_schedule_in_print` used but never assigned in __init__: M…	`erpnext/accounts/doctype/accounts_setti…:135`
HIGH	`MINED108`	[MINED108] `self.update_property_for_accounting_dimension` used but never assigned in __i…	`erpnext/accounts/doctype/accounts_setti…:157`
HIGH	`MINED108`	[MINED108] `self.validate_and_sync_auto_reconcile_config` used but never assigned in __in…	`erpnext/accounts/doctype/accounts_setti…:156`
HIGH	`MINED108`	[MINED108] `self.enable_subscription` used but never assigned in __init__: Method `valida…	`erpnext/accounts/doctype/accounts_setti…:149`
HIGH	`MINED108`	[MINED108] `self.enable_loyalty_point_program` used but never assigned in __init__: Metho…	`erpnext/accounts/doctype/accounts_setti…:145`
HIGH	`MINED108`	[MINED108] `self.enable_discounts_and_margin` used but never assigned in __init__: Method…	`erpnext/accounts/doctype/accounts_setti…:141`
HIGH	`MINED108`	[MINED108] `self.enable_accounting_dimensions` used but never assigned in __init__: Metho…	`erpnext/accounts/doctype/accounts_setti…:137`
HIGH	`MINED108`	[MINED108] `self.show_payment_schedule_in_print` used but never assigned in __init__: Met…	`erpnext/accounts/doctype/accounts_setti…:134`
HIGH	`MINED108`	[MINED108] `self.validate_stale_days` used but never assigned in __init__: Method `valida…	`erpnext/accounts/doctype/accounts_setti…:132`
HIGH	`MINED108`	[MINED108] `self.enable_common_party_accounting` used but never assigned in __init__: Met…	`erpnext/accounts/doctype/accounts_setti…:126`
HIGH	`MINED108`	[MINED108] `self.add_taxes_from_item_tax_template` used but never assigned in __init__: M…	`erpnext/accounts/doctype/accounts_setti…:120`
HIGH	`MINED108`	[MINED108] `self.get_doc_before_save` used but never assigned in __init__: Method `valida…	`erpnext/accounts/doctype/accounts_setti…:117`
HIGH	`MINED108`	[MINED108] `self.validate_auto_tax_settings` used but never assigned in __init__: Method …	`erpnext/accounts/doctype/accounts_setti…:116`
HIGH	`MINED108`	[MINED108] `self.as_dict` used but never assigned in __init__: Method `on_update` of clas…	`erpnext/accounts/custom/address.py:47`
HIGH	`MINED108`	[MINED108] `self.name` used but never assigned in __init__: Method `on_update` of class `…	`erpnext/accounts/custom/address.py:48`
HIGH	`MINED108`	[MINED108] `self.links` used but never assigned in __init__: Method `validate_reference` …	`erpnext/accounts/custom/address.py:31`
HIGH	`MINED108`	[MINED108] `self.is_your_company_address` used but never assigned in __init__: Method `va…	`erpnext/accounts/custom/address.py:31`
HIGH	`MINED108`	[MINED108] `self.is_your_company_address` used but never assigned in __init__: Method `up…	`erpnext/accounts/custom/address.py:28`
HIGH	`MINED108`	[MINED108] `self.get` used but never assigned in __init__: Method `update_company_address…	`erpnext/accounts/custom/address.py:26`
HIGH	`MINED108`	[MINED108] `self.is_your_company_address` used but never assigned in __init__: Method `li…	`erpnext/accounts/custom/address.py:20`
HIGH	`MINED108`	[MINED108] `self.update_company_address` used but never assigned in __init__: Method `val…	`erpnext/accounts/custom/address.py:13`
HIGH	`MINED108`	[MINED108] `self.validate_reference` used but never assigned in __init__: Method `validat…	`erpnext/accounts/custom/address.py:12`
HIGH	`MINED001`	[MINED001] Bare Except Pass: except: pass or except Exception: pass — silently swallows e…	`erpnext/accounts/doctype/bank_transacti…:117`
HIGH	`MINED009`	[MINED009] Floats For Money: Variable named price/amount/cost typed as float instead of D…	`erpnext/accounts/doctype/bank_account/b…:191`
HIGH	`SEC128`	[SEC128] Async function without await — fire-and-forget Promise (AI mistake): Async call …	`erpnext/accounts/doctype/accounting_dim…:52`
HIGH	`SEC128`	[SEC128] Async function without await — fire-and-forget Promise (AI mistake): Async call …	`erpnext/accounts/doctype/account_closin…:56`
HIGH	`SEC128`	[SEC128] Async function without await — fire-and-forget Promise (AI mistake): Async call …	`erpnext/__init__.py:64`
HIGH	`SEC078`	[SEC078] Python: requests without timeout: requests.get/post without a timeout will hang …	`.github/helper/documentation.py:44`
HIGH	`SEC029`	[SEC029] Server-Side Request Forgery (SSRF) — outbound HTTP from user input: Outbound HTT…	`.github/helper/documentation.py:18`
HIGH	`MINED126`	[MINED126] Workflow container/services image `mariadb:10.6` unpinned: `container/services…	`.github/workflows/server-tests-mariadb.…:58`
HIGH	`MINED126`	[MINED126] Workflow container/services image `postgres:13.3` unpinned: `container/service…	`.github/workflows/server-tests-postgres…:36`
HIGH	`MINED115`	[MINED115] Action `actions/setup-node` pinned to mutable ref `@v6`: `uses: actions/setup-…	`.github/workflows/server-tests-postgres…:66`
HIGH	`MINED115`	[MINED115] Action `actions/setup-python` pinned to mutable ref `@v6`: `uses: actions/setu…	`.github/workflows/server-tests-postgres…:53`
HIGH	`MINED115`	[MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout…	`.github/workflows/server-tests-postgres…:50`
HIGH	`MINED126`	[MINED126] Workflow container/services image `alpine:latest` unpinned: `container/service…	`.github/workflows/docker-release.yml:13`
HIGH	`MINED115`	[MINED115] Action `actions/setup-node` pinned to mutable ref `@v6`: `uses: actions/setup-…	`.github/workflows/generate-pot-file.yml:34`
HIGH	`MINED115`	[MINED115] Action `actions/setup-python` pinned to mutable ref `@v6`: `uses: actions/setu…	`.github/workflows/generate-pot-file.yml:29`
HIGH	`MINED115`	[MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout…	`.github/workflows/generate-pot-file.yml:24`
HIGH	`MINED115`	[MINED115] Action `actions/setup-node` pinned to mutable ref `@v6`: `uses: actions/setup-…	`.github/workflows/semantic-commits.yml:21`
HIGH	`MINED115`	[MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout…	`.github/workflows/semantic-commits.yml:18`
HIGH	`MINED115`	[MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout…	`.github/workflows/docs-checker.yml:21`
HIGH	`MINED115`	[MINED115] Action `actions/setup-python` pinned to mutable ref `@v6`: `uses: actions/setu…	`.github/workflows/docs-checker.yml:16`
HIGH	`MINED115`	[MINED115] Action `actions/labeler` pinned to mutable ref `@v3`: `uses: actions/labeler@v…	`.github/workflows/labeller.yml:14`
HIGH	`MINED126`	[MINED126] Workflow container/services image `mariadb:11.8` unpinned: `container/services…	`.github/workflows/patch.yml:32`
HIGH	`MINED115`	[MINED115] Action `actions/cache` pinned to mutable ref `@v4`: `uses: actions/cache@v4` r…	`.github/workflows/patch.yml:92`
HIGH	`MINED115`	[MINED115] Action `actions/cache` pinned to mutable ref `@v4`: `uses: actions/cache@v4` r…	`.github/workflows/patch.yml:78`
HIGH	`MINED115`	[MINED115] Action `actions/cache` pinned to mutable ref `@v4`: `uses: actions/cache@v4` r…	`.github/workflows/patch.yml:69`
HIGH	`MINED115`	[MINED115] Action `actions/setup-node` pinned to mutable ref `@v6`: `uses: actions/setup-…	`.github/workflows/patch.yml:60`
HIGH	`MINED115`	[MINED115] Action `actions/setup-python` pinned to mutable ref `@v6`: `uses: actions/setu…	`.github/workflows/patch.yml:52`
HIGH	`MINED115`	[MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout…	`.github/workflows/patch.yml:41`
HIGH	`MINED126`	[MINED126] Workflow container/services image `mariadb:10.6` unpinned: `container/services…	`.github/workflows/run-individual-tests.…:67`
HIGH	`MINED115`	[MINED115] Action `actions/cache` pinned to mutable ref `@v4`: `uses: actions/cache@v4` r…	`.github/workflows/run-individual-tests.…:116`
HIGH	`MINED115`	[MINED115] Action `actions/cache` pinned to mutable ref `@v4`: `uses: actions/cache@v4` r…	`.github/workflows/run-individual-tests.…:102`
HIGH	`MINED115`	[MINED115] Action `actions/cache` pinned to mutable ref `@v4`: `uses: actions/cache@v4` r…	`.github/workflows/run-individual-tests.…:93`
HIGH	`MINED115`	[MINED115] Action `actions/setup-node` pinned to mutable ref `@v6`: `uses: actions/setup-…	`.github/workflows/run-individual-tests.…:84`
HIGH	`MINED115`	[MINED115] Action `actions/setup-python` pinned to mutable ref `@v6`: `uses: actions/setu…	`.github/workflows/run-individual-tests.…:79`
HIGH	`MINED115`	[MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout…	`.github/workflows/run-individual-tests.…:76`
HIGH	`MINED115`	[MINED115] Action `actions/checkout` pinned to mutable ref `@v6`: `uses: actions/checkout…	`.github/workflows/run-individual-tests.…:20`
HIGH	`MINED115`	[MINED115] Action `actions/github-script` pinned to mutable ref `@v7`: `uses: actions/git…	`.github/workflows/label-base-on-title.y…:15`
MED	`MINED111`	[MINED111] Bare except continues silently: Bare `except:` (or `except Exception:`) that r…	`erpnext/accounts/doctype/bank_statement…:162`
MED	`CFG006`	[CFG006] Missing .gitignore: No .gitignore file. Risk of committing secrets and build art…	—
MED	`ERR001`	[ERR001] Silent Exception Swallowing: Silently swallowing all exceptions hides bugs. Even…	`erpnext/accounts/doctype/bank_transacti…:117`
MED	`SEC015`	[SEC015] Insecure Randomness for Security: Weak PRNG used in security-sensitive context. …	`erpnext/accounts/doctype/account_closin…:93`
MED	`SEC041`	[SEC041] Tabnabbing — target="_blank" without rel="noopener noreferrer": <a target="_blan…	`erpnext/accounts/doctype/bank_statement…:8`
MED	`SEC041`	[SEC041] Tabnabbing — target="_blank" without rel="noopener noreferrer": <a target="_blan…	`banking/src/pages/BankStatementImporter…:237`
MED	`SEC041`	[SEC041] Tabnabbing — target="_blank" without rel="noopener noreferrer": <a target="_blan…	`banking/src/components/common/LinkField…:246`
MED	`COMP001`	[COMP001] High cognitive complexity: Function `get_chart` has cognitive complexity 20 (So…	`erpnext/accounts/doctype/account/chart_…:102`
MED	`AUC001`	[AUC001] No Repobility access matrix policy found: The repository uses web/API frameworks…	—
LOW	`COMP001`	[COMP001] High cognitive complexity: Function `build_result` has cognitive complexity 12 …	`erpnext/accounts/dashboard_chart_source…:70`
LOW	`COMP001`	[COMP001] High cognitive complexity: Function `get` has cognitive complexity 13 (SonarSou…	`erpnext/accounts/dashboard_chart_source…:15`
LOW	`AIC003`	Duplicated implementation block across source files	`banking/src/components/ui/textarea.tsx:12`
LOW	`AIC003`	Duplicated implementation block across source files	`banking/src/components/ui/select.tsx:24`
LOW	`AIC003`	Duplicated implementation block across source files	`banking/src/components/features/BankRec…:180`
LOW	`AIC003`	Duplicated implementation block across source files	`banking/src/components/features/BankRec…:444`
LOW	`AIC003`	Duplicated implementation block across source files	`banking/src/components/features/BankRec…:137`
INFO	`MINED044`	[MINED044] Js Console Log Prod: console.log left in code. Should be replaced with logger …	`erpnext/accounts/doctype/bank/bank.js:116`
INFO	`MINED072`	[MINED072] Python Pass Only Class: class Foo: pass — stub waiting to be filled in.	`erpnext/accounts/doctype/accounting_per…:10`
INFO	`MINED050`	[MINED050] Stub Only Function: Function declared but body is just pass, return None, rais…	`erpnext/accounts/doctype/accounting_per…:11`
INFO	`MINED050`	[MINED050] Stub Only Function: Function declared but body is just pass, return None, rais…	`erpnext/accounts/doctype/accounting_dim…:29`
INFO	`MINED050`	[MINED050] Stub Only Function: Function declared but body is just pass, return None, rais…	`erpnext/accounts/doctype/account_closin…:42`
INFO	`MINED052`	[MINED052] Ts Any Typed: : any used as type annotation. Defeats TypeScript type safety.	`banking/src/hooks/useDocType.ts:15`
INFO	`MINED045`	[MINED045] Ts Non Null Assertion: x! asserts not null - bypasses null checks - TypeError …	`banking/src/components/ui/dropdown-menu…:63`
INFO	`MINED056`	[MINED056] React Key As Index: key={index} in map() — re-renders the wrong elements on re…	`banking/src/components/ui/loaders.tsx:9`
INFO	`MINED056`	[MINED056] React Key As Index: key={index} in map() — re-renders the wrong elements on re…	`banking/src/components/ui/error-banner.…:44`
INFO	`MINED056`	[MINED056] React Key As Index: key={index} in map() — re-renders the wrong elements on re…	`banking/src/components/features/BankSta…:41`
INFO	`MINED058`	[MINED058] React Dangerously Set Html: dangerouslySetInnerHTML bypasses Reacts JSX escapi…	`banking/src/components/features/BankRec…:184`
INFO	`MINED043`	[MINED043] Http Not Https: Hardcoded http:// (not localhost) for endpoints that handle cr…	`banking/proxyOptions.ts:10`
INFO	`MINED067`	[MINED067] Python Requests No Timeout: requests.get/post/etc. without timeout= can hang f…	`.github/helper/documentation.py:44`

Reset to top 5 92 findings available (after auto-suppression of test files + won't-fix)

Issue body (markdown)

## Code-quality scan: `frappe/erpnext`

**Score: 74/100 (B-)**  ·  95 findings  ·  scanned 2026-05-20 01:10 UTC  ·  35,376 LOC

| Severity | Count |
|---|---|
| CRITICAL | 1 |
| HIGH | 62 |
| MEDIUM | 9 |
| LOW | 7 |

📊 [Full filterable report](https://repobility.com/scan/7bfa3179-7646-4f6c-9965-6c3558bc8cc9/)  ·  ![scorecard](https://repobility.com/scan/7bfa3179-7646-4f6c-9965-6c3558bc8cc9/report.png?v=1779239448-s2)

### Top findings

1. **CRITICAL** `MINED116` — Workflow uses `secrets.CODECOV_TOKEN` on a `pull_request` trigger
   `.github/workflows/server-tests-mariadb.yml:164` · ✓ Repobility
2. **HIGH** `MINED108` — `self.enable_accounting_dimensions` used but never assigned in __init__
   `erpnext/accounts/doctype/accounts_settings/accounts_settings.py:138` · ✓ Repobility
3. **HIGH** `MINED108` — `self.get` used but never assigned in __init__
   `erpnext/accounts/doctype/accounts_settings/accounts_settings.py:128` · ✓ Repobility
4. **HIGH** `MINED108` — `self.get` used but never assigned in __init__
   `erpnext/accounts/doctype/accounts_settings/accounts_settings.py:122` · ✓ Repobility
5. **HIGH** `MINED108` — `self.enable_payment_schedule_in_print` used but never assigned in __init__
   `erpnext/accounts/doctype/accounts_settings/accounts_settings.py:135` · ✓ Repobility

---

_Filed automatically. Close this issue if not useful — we won't refile. Full report: https://repobility.com/scan/7bfa3179-7646-4f6c-9965-6c3558bc8cc9/_

Already filed

This repo publishes a SECURITY.md policy and the scan contains 5 Critical/High security finding(s). Public issue filing would violate coordinated disclosure. Submit privately via the project's security reporting channel.

Megaproject â high spam risk

Could not determine 'frappe/erpnext' star count (GitHub API rate-limited or unreachable). When in doubt about repo size, prefer opening a focused PR or a discussion rather than an issue.

Already filed

37/99 findings (37%) on this scan are already flagged as test-file, won't-fix, or suppressed. The scan is too noisy to file as a single issue. Curate down to specific actionable findings, or address the FP source first.

Open in GitHub to file this issue Filing guard overridden Cancel

The button opens GitHubâs new-issue page in a new tab. You will see the title + body pre-filled â review, edit if you want, then click GitHubâs "Submit new issue" button. Repobility never posts anything on your behalf.

For real security findings on big repos: use the project's SECURITY.md or private advisory flow instead of a public issue.

Push this scan report to frappe/erpnext

Embedded score card image

Issue title

Curate findings to include

Issue body (markdown)

Push this scan report to `frappe/erpnext`