- Issue created by @CNDexter
- Status changed to Needs review
10 months ago 11:05am 22 May 2024 - 🇮🇳India sarwan_verma
Hi @CNDexter,
I have fixed this issue "Undefined array key real_aes" and also attached patch ,
please review and verify. - First commit to issue fork.
- 🇮🇳India manish-31
Hi @sarwan_verma,
IMO we should not fix this by just adding an isset() check the warning occurs only when Real AES module is installed and the checkbox gets removed from the checklist options. Which looks wrong to me.
As in other cases, we are showing the project link if a module is missing and also showing settings/configuration link if the module is installed we should replicate the same behaviour here.
I have raised an MR to implement the same. Attached Screenshots of the UI for both cases Real AES enabled and disabled. Needs review.
Tested MR15 , the issue is resolved and Manage Keys option is also showing when real aed module is installed . Attached a video for same. RTBC++.
- Status changed to RTBC
8 months ago 11:16am 4 July 2024 - First commit to issue fork.
-
thejimbirch →
committed 23e410d4 on 1.x authored by
manish-31 →
Issue #3448908 by manish-31, sarwan_verma, sourabhsisodia_, cndexter,...
-
thejimbirch →
committed 23e410d4 on 1.x authored by
manish-31 →
- Status changed to Fixed
about 1 month ago 3:27pm 30 January 2025 - 🇺🇸United States thejimbirch Cape Cod, Massachusetts
Merging to dev. Thanks for the contribution!
Automatically closed - issue fixed for 2 weeks with no activity.