- Issue created by @CNDexter
- Status changed to Needs review
7 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
6 months ago 11:16am 4 July 2024