Skip to content

Commit 4cf3537

Browse files
author
felix.manrique
committed
feature: modified UX experience for whitelisting plugins
1 parent b94ca6e commit 4cf3537

13 files changed

Lines changed: 167 additions & 80 deletions

File tree

CHANGELOG.md

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,11 +4,22 @@
44
### The Post Revision Update
55

66
### Fixed
7+
- **Dashboard Tab Navigation:** Fixed tabbed navigation in dashboard not switching between sections (Overview, Dependency Audit, Role Heatmap, etc.). Replaced Bootstrap 5 attributes (`data-bs-toggle`, `data-bs-target`) with Bootstrap 4 syntax (`data-toggle`, `data-target`) to match Moodle's Bootstrap version.
8+
- **Self-Scanning Exclusion:** The plugin now explicitly excludes itself (`local_mrca`) from risk scans to avoid the ironic situation of the risk auditor flagging itself as "Critical" risk. The scanner now skips self-analysis before processing other plugins.
79
- **Hard-coded Language Strings:** Replaced all hard-coded user-facing text with proper `get_string()` calls for full internationalization support. Added 27 new language strings for CSV report headers, deprecated function descriptions, and unsafe function warnings across all supported languages (en, es, fr, it, pt).
810
- **Non-English Comments:** Translated all Spanish comments in `db/upgrade.php` to English for international collaboration compliance.
911
- **Database Performance (N+1 Queries):** Optimized `dashboard.php` role heatmap generation by preloading all roles in a single bulk query using `get_in_or_equal()`, eliminating N+1 query problem in role risk display.
1012
- **Third-Party Library Documentation:** Created `thirdpartylibs.xml` to properly document Chart.js v4.5.1 (MIT License) as required by Moodle plugin guidelines.
1113

14+
### Improved
15+
- **Whitelist UX:** Improved the user experience for whitelisting PII fields:
16+
- Added visual badge showing number of detected PII fields next to database icon
17+
- New "Whitelist All X Fields" button to mark all fields as safe at once
18+
- Individual whitelist buttons now use check-circle icon and green color for better visibility
19+
- Improved tooltips with clearer action descriptions
20+
- Better visual hierarchy with styled field badges and spacing
21+
- Expanded PII section shows field count and bulk actions prominently
22+
1223
### Changed
1324
- **Plugin Name:** Rebranded from "Moodle Risk & Compliance Analyzer" to "Risk & Compliance Analyzer for Moodle" across all language files and CLI scripts for better naming consistency.
1425
- **CSV Report Generation:** All CSV headers and labels now use language strings from `get_string()` instead of hard-coded English text, enabling proper translation.

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88

99
MRCA is a multi-layered risk analysis engine for Moodle™. It scans installed plugins for security risks, privacy compliance gaps, outdated dependencies, and permission misconfigurations — providing administrators with a unified **Site Risk Index** and actionable alerts.
1010

11-
By default, MRCA only scans **third-party plugins**. Core Moodle modules are excluded to avoid false positives.
11+
By default, MRCA only scans **third-party plugins**. Core Moodle modules and the MRCA plugin itself are excluded to avoid false positives and self-flagging.
1212

1313
## Why MRCA?
1414

amd/build/dashboard.min.js

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

amd/build/dashboard.min.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)