First, I'll apologize for failing to see patch 3.
That said, I applied it, and the error changes now to:
Warning
: Undefined array key "upn" in
Drupal\openid_connect_windows_aad\Plugin\OpenIDConnectClient\WindowsAad->buildUserinfo()
(line
475
of
modules/contrib/openid_connect_windows_aad/src/Plugin/OpenIDConnectClient/WindowsAad.php
)
Not sure if that belongs in a new ticket since it's a different error now.
This is actually the last item preventing us from completing a migration to D10, so I think the priority needs to be increased.
srees → created an issue.
I do not understand how this cannot be a bug. Did you try my steps to reproduce? The only "custom" thing I'm doing is taking advantage of the option within the attached PDF field to generate output from embedded TWIG.
The twig template is receiving the data in the order it is set in the webform. This is expected. However viewing a link to the PDF is in order of key, which is not expected. These should be uniform without me having to customize through a hook.
I'm not even asking that the TWIG template be honored in the online version of the PDF (though that would be logical). Just that data be provided in a consistent format.
I'm not sure this is the best way to solve it, but I've written a patch that is working for me.