SpelExpression throws NullPointerException instead of EvaluationException for primitives [SPR-16123]

[spring-projects-issues]

Wojciech Gruszczyk opened SPR-16123 and commented

Bug proof:

package com.demo.bugproof;

import org.junit.Test;
import org.springframework.expression.EvaluationException;
import org.springframework.expression.spel.standard.SpelExpressionParser;


public class SpelNPEDemoTest
{

	@Test(expected = EvaluationException.class)
	public void npeBugProof()
	{
		new SpelExpressionParser().parseExpression("demo").setValue(new SomePOJO(), null);
	}

	private class SomePOJO
	{
		private boolean demo;

		public boolean isDemo()
		{
			return demo;
		}

		public void setDemo(final boolean demo)
		{
			this.demo = demo;
		}
	}

}

---

Affects: 4.3.11

Issue Links:

• Backport selected refinements from the nullability efforts in 5.0 [SPR-15656] #20215 Backport selected refinements from the nullability efforts in 5.0

Referenced from: commits fd6f2bd

[spring-projects-issues]

Juergen Hoeller commented

This works fine against 5.0.x for me; I'll try against 4.3.x tomorrow.

[spring-projects-issues]

Juergen Hoeller commented

As far as I can tell, this works against 4.3.12 already, assumably as a side effect of #20215 (a wider backport of nullability refinements from 5.0). Could you please doube-check against 4.3.12?

[spring-projects-issues]

Wojciech Gruszczyk commented

I confirm. On 4.3.12 it works as expected. Thank you for quick feedback!