Typo in error message

Daemach · June 12, 2019, 7:00pm

Should be “too large”…

Lucee 5.3.2.77 Error (database)
Message value [9718088504] is to large, defined maxlength is [10] but length of value is [11]
Stacktrace The Error Occurred in
C:\Dropbox\Private_Sites\THCGuard\wwwroots\Main\handleverifymobile.cfm: line 7

5: SELECT *
6: FROM Users
7: WHERE mobile = AND
8:
9: mobileVerifyCode2 =

Zackster · June 12, 2019, 11:00pm

Wanna file a PR on GitHub against the 5.3 branch? It’s super easy

github.com

lucee/Lucee/blob/5.3/core/src/main/java/lucee/runtime/tag/QueryParam.java#L240


      
          		}
          		return SKIP_BODY;
          	}
          
          
	public static Object check(Object value, int type, int maxlength, Charset charset) throws PageException {
          		if (maxlength != -1 || charset != null) {
          
          
			String str;
          			if (BIGINT == type || INTEGER == type || SMALLINT == type || TINYINT == type) {
          				str = Caster.toString(Caster.toIntValue(value));
          			}
          			else if (BOOLEAN == type) {
          				str = Caster.toString(Caster.toBooleanValue(value));
          			}
          			else if (DECIMAL == type) {
          				str = Caster.toDecimal(value, false);
          			}
          			else str = Caster.toString(value);
          
          
			if (charset != null) {
          				if (!StringUtil.isCompatibleWith(str, charset)) throw new DatabaseException(

cfmitrah · June 17, 2019, 11:45am

I’ve changed the error message along with this PR : Added a feature for LDEV-2315 & Changed the error message in Queryparam by cfmitrah · Pull Request #698 · lucee/Lucee · GitHub