X-Git-Url: https://vcs.fsf.org/?a=blobdiff_plain;f=ext%2Fafform%2Fdocs%2Froadmap.md;h=3b0b0c6b1fc07d32f6219560a9163aac2aa1d94f;hb=7d41ce47ba7266e03f8baf39482c0e7389bfe30c;hp=ab8639a1fc4996d4a66dfaa06a0b2414de4422b3;hpb=954703502f5b850dc3fb035b54365d01dc44bf9c;p=civicrm-core.git diff --git a/ext/afform/docs/roadmap.md b/ext/afform/docs/roadmap.md index ab8639a1fc..3b0b0c6b1f 100644 --- a/ext/afform/docs/roadmap.md +++ b/ext/afform/docs/roadmap.md @@ -20,11 +20,16 @@ This extension is expected to be the base for a suite of related extensions: * `afform_auditor`: Report on upgrade compatibility. Test changesets. Highlight dangerous/unknown/unsupported form elements. Score maintainability of the system. +## Documentation + +* Development and Refactoring: A guide for making changes to supported markup in a maintainable fashion. + (*What if I need to rename a tag? What if I need to deprecate a tag? Ad nauseum*) + ## Known Issues Within this extension, there are things which need updating/addressing: -* Test coverage for key Angular directives (e.g. `aff-api4-ctrl`, `aff-api4-action`) +* Test coverage for key Angular directives (e.g. `af-api4-ctrl`, `af-api4-action`) * There are several `FIXME`/`TODO` declarations in the code for checking pre-conditions, reporting errors, handling edge-cases, etc. * Although afforms can be used in AngularJS, they don't fully support tooling like `cv ang:html:list` and `hook_civicrm_alterAngular` changesets. We'll need a core patch to allow that. (Ex: Define partials via callback.) @@ -45,3 +50,7 @@ Within this extension, there are things which need updating/addressing: support batching (i.e. all API calls issued within a 5ms window are sent as a batch). * Default CSS: There's no mechanism for defining adhoc CSS. This is arguably a feature, though, because the CSS classes should be validated (to ensure theme interoperability). +* `Civi/Angular/ChangeSet.php` previously had an integrity check that activated in developer mode + (`\CRM_Core_Config::singleton()->debug && $coder->checkConsistentHtml($html)`). This has been removed because it was a bit brittle + about self-closing HTML tags. However, the general concept of HTML validation should be reinstated as part of the `afform_auditor`. +* `hook_alterAngular` is used to inject APIv4 metadata for certain tags. This behavior needs a unit-test.