Part 2 of prepping for #5151; minor change post #3072 (signature change) (#5155)

Author: cowtowncoder

src/main/java/com/fasterxml/jackson/databind/DeserializationContext.java

@@ -478,7 +478,7 @@ public final Object findInjectableValue(Object valueId,
 "No 'injectableValues' configured, cannot inject value with id '%s'", valueId),
                     valueId, forProperty, beanInstance);
         }
-        return _injectableValues.findInjectableValue(valueId, this, forProperty, beanInstance, optional);
+        return _injectableValues.findInjectableValue(this, valueId, forProperty, beanInstance, optional);
     }
 
     /**

src/main/java/com/fasterxml/jackson/databind/InjectableValues.java

@@ -17,17 +17,19 @@ public abstract class InjectableValues
      * (will be available when injected via field or setter; not available
      * when injected via constructor or factory method argument).
      *
+     * @param ctxt Deserialization context
      * @param valueId Object that identifies value to inject; may be a simple
      *   name or more complex identifier object, whatever provider needs
-     * @param ctxt Deserialization context
      * @param forProperty Bean property in which value is to be injected
      * @param beanInstance Bean instance that contains property to inject,
      *    if available; null if bean has not yet been constructed.
      * @param optional Flag used for configuring the behavior when the value
      *    to inject is not found
      */
-    public Object findInjectableValue(Object valueId, DeserializationContext ctxt,
-            BeanProperty forProperty, Object beanInstance, Boolean optional) throws JsonMappingException
+    public Object findInjectableValue(DeserializationContext ctxt,
+            Object valueId, BeanProperty forProperty, Object beanInstance,
+            Boolean optional)
+        throws JsonMappingException
     {
         // For backwards-compatibility, must delegate to old method
         return findInjectableValue(valueId, ctxt, forProperty, beanInstance);
@@ -80,7 +82,7 @@ public Std addValue(Class<?> classKey, Object value) {
          * @since 2.20
          */
         @Override
-        public Object findInjectableValue(Object valueId, DeserializationContext ctxt,
+        public Object findInjectableValue(DeserializationContext ctxt, Object valueId,
                 BeanProperty forProperty, Object beanInstance, Boolean optional)
             throws JsonMappingException
         {
@@ -113,7 +115,7 @@ public Object findInjectableValue(Object valueId, DeserializationContext ctxt,
         public Object findInjectableValue(Object valueId, DeserializationContext ctxt,
                 BeanProperty forProperty, Object beanInstance) throws JsonMappingException
         {
-            return this.findInjectableValue(valueId, ctxt, forProperty, beanInstance, null);
+            return this.findInjectableValue(ctxt, valueId, forProperty, beanInstance, null);
         }
     }
 }

src/test/java/com/fasterxml/jackson/databind/introspect/IntrospectorPairTest.java

@@ -697,7 +697,8 @@ public JacksonInject.Value findInjectableValue(AnnotatedMember m) {
 
     static class TestInjector extends InjectableValues {
         @Override
-        public Object findInjectableValue(Object valueId, DeserializationContext ctxt,
+        public Object findInjectableValue(DeserializationContext ctxt,
+                Object valueId, 
                 BeanProperty forProperty, Object beanInstance, Boolean optional) {
             if (valueId == "jjj") {
                 UnreadableBean bean = new UnreadableBean();

src/test/java/com/fasterxml/jackson/databind/tofix/JacksonInject4218Test.java

@@ -36,16 +36,16 @@ class MyInjectableValues extends InjectableValues.Std
 
         @Override
         public Object findInjectableValue(
-                Object valueId,
                 DeserializationContext ctxt,
+                Object valueId,
                 BeanProperty forProperty,
                 Object beanInstance,
                 Boolean optional
         ) throws JsonMappingException {
             if (valueId.equals("id")) {
                 return "id" + nextId++;
             } else {
-                return super.findInjectableValue(valueId, ctxt, forProperty, beanInstance, optional);
+                return super.findInjectableValue(ctxt, valueId, forProperty, beanInstance, optional);
             }
         }
     }