Add options to unused list when unknown key is active

Author: josephwright

base/changes.txt

@@ -6,6 +6,10 @@ to completeness or accuracy and it contains some references to files that are
 not part of the distribution.
 ================================================================================
 
+2024-06-03  Joseph Wright  <[email protected]>
+	* ltkeys.dtx:
+	Correct addition to unused option list with unknown key property (gh/1183)
+
 2024-06-19  David Carlisle  <[email protected]>
 
 	* fontdef.dtx:

base/ltkeys.dtx

@@ -33,7 +33,7 @@
 %<*driver>
 % \fi
 \ProvidesFile{ltkeys.dtx}
-             [2024/06/20 v1.0o LaTeX Kernel (Keyval options)]
+             [2024/06/25 v1.0p LaTeX Kernel (Keyval options)]
 % \iffalse
 \documentclass{l3doc}
 \GetFileInfo{ltkeys.dtx}
@@ -361,6 +361,8 @@
 % \changes{v1.0h}{2022/06/20}{Use raw options data}
 % \changes{v1.0m}{2024/01/13}{Trim spaces off key names}
 % \changes{v1.0n}{2024/06/19}{Refactor function}
+% \changes{v1.0p}{2024/06/25}
+%   {Correct unused option tracking where unknown keys are allowed}
 % \begin{macro}{\@@_options_class:nnn}
 % \changes{v1.0h}{2022/06/20}{New function}
 % \changes{v1.0i}{2022/07/05}{Correct naming of raw class options storage}
@@ -387,7 +389,7 @@
           {
             \exp_args:Ne \@@_options_class:nnn
               { \tl_trim_spaces:e { \@@_remove_equals:n {##1} } }
-                {##1} {#1}
+              {##1} {#1}
           }
       }
   }
@@ -400,7 +402,7 @@
       }
       {
         \keys_if_exist:nnTF {#3} { unknown }
-          { \@@_options_class:nn {#1} {#2} }
+          { \clist_put_right:Nn \l_@@_options_clist {#2} }
           {
             \clist_if_in:NnF \@unusedoptionlist {#1}
               { \clist_put_right:Nn \@unusedoptionlist {#1} }

base/testfiles/github-1183.lvt

@@ -0,0 +1,28 @@
+\input{test2e}
+
+\begin{filecontents}[overwrite]{github-1183.cls}
+\NeedsTeXFormat{LaTeX2e}
+\ProvidesClass{github-1183}
+\LoadClassWithOptions{article}
+
+\DeclareKeys[testwork]{%
+  test .code  = \newcommand{\foo}{#1},
+  test .usage = load,
+  % the following three lines give an equivalent definition of your bertha
+  % option as defined with `\DeclareOption{bertha}{}`
+  bertha .code = {},
+  bertha .value_forbidden:n = true,
+  bertha .usage = load,
+  % if you use this unknown handler, you get "test" and "bertha" as unknown
+  % options, if you omit it you get "a4paper" as unknown option
+  unknown .code = {}
+}
+\ProcessKeyOptions[testwork]
+
+\endinput
+\end{filecontents}
+\documentclass[a4paper,test=wtf,bertha]{github-1183}
+
+\START
+\begin{document}
+\END

base/testfiles/github-1183.tlg

@@ -0,0 +1,17 @@
+This is a generated file for the LaTeX2e validation system.
+Don't change this file in any respect.
+(github-1183.aux)
+LaTeX Font Info:    Checking defaults for OML/cmm/m/it on input line ....
+LaTeX Font Info:    ... okay on input line ....
+LaTeX Font Info:    Checking defaults for OMS/cmsy/m/n on input line ....
+LaTeX Font Info:    ... okay on input line ....
+LaTeX Font Info:    Checking defaults for OT1/cmr/m/n on input line ....
+LaTeX Font Info:    ... okay on input line ....
+LaTeX Font Info:    Checking defaults for T1/cmr/m/n on input line ....
+LaTeX Font Info:    ... okay on input line ....
+LaTeX Font Info:    Checking defaults for TS1/cmr/m/n on input line ....
+LaTeX Font Info:    ... okay on input line ....
+LaTeX Font Info:    Checking defaults for OMX/cmex/m/n on input line ....
+LaTeX Font Info:    ... okay on input line ....
+LaTeX Font Info:    Checking defaults for U/cmr/m/n on input line ....
+LaTeX Font Info:    ... okay on input line ....