Minimal length isn't set for text fields

4 views
Skip to first unread message

Alexander Obuhovich

unread,
Aug 18, 2011, 2:26:48 PM8/18/11
to In-Portal Bugs
In-Portal automatic unit config generator based on database doesn't set minimal field length to 0 symbols.

Because of it error message "Field value length is out of range, possible value length from to 3" looks strange.

When minimal and maximal field length is set, then message would look like "Field value length is out of range, possible value length from 0 to 3" (0 added).


--
Best Regards,

http://www.in-portal.com
http://www.alex-time.com

Phil -- wbtc.fr --

unread,
Aug 18, 2011, 6:17:30 PM8/18/11
to in-port...@googlegroups.com
interesting catch... But are we only 2 in groups now?

2011/8/18 Alexander Obuhovich <aik....@gmail.com>

--
You received this message because you are subscribed to the Google Groups "In-Portal Bugs Team" group.
To post to this group, send email to in-port...@googlegroups.com.
To unsubscribe from this group, send email to in-portal-bug...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/in-portal-bugs?hl=en.

Alexander Obuhovich

unread,
Aug 19, 2011, 3:38:55 AM8/19/11
to in-port...@googlegroups.com
Nope, Dmitry is answering from time to time.

Dmitry A.

unread,
Aug 20, 2011, 12:15:21 PM8/20/11
to in-port...@googlegroups.com
Do you use that Config Generator anywhere? I thought it's still in development/beta?


DA

Alexander Obuhovich

unread,
Aug 20, 2011, 12:16:17 PM8/20/11
to in-port...@googlegroups.com
I've talked about function in "System Tools", that create a set of unit config settings based on given table database structure.


On Sat, Aug 20, 2011 at 7:15 PM, Dmitry A. <dand...@gmail.com> wrote:
Do you use that Config Generator anywhere? I thought it's still in development/beta?


DA

--
You received this message because you are subscribed to the Google Groups "In-Portal Bugs Team" group.
To view this discussion on the web visit https://groups.google.com/d/msg/in-portal-bugs/-/cfkuxVZCJ-MJ.

To post to this group, send email to in-port...@googlegroups.com.
To unsubscribe from this group, send email to in-portal-bug...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/in-portal-bugs?hl=en.

Dmitry A.

unread,
Aug 20, 2011, 5:24:50 PM8/20/11
to in-port...@googlegroups.com
Thanks for clarifying Alex.

Yes, we should correct this is there are no other side-affects related to the change.


DA

Alexander Obuhovich

unread,
Mar 26, 2012, 6:50:03 AM3/26/12
to in-port...@googlegroups.com
On the other hand I would rather display 0 in error message, when 'min_len' option isn't set, because I see no point in forcing developers to set 'min_len' => 0 for all fields, that have 'max_len' option set, when it's 0 anyway.



Ready for testing.


--
You received this message because you are subscribed to the Google Groups "In-Portal Bugs Team" group.
To view this discussion on the web visit https://groups.google.com/d/msg/in-portal-bugs/-/HjgOg2F5hu4J.

To post to this group, send email to in-port...@googlegroups.com.
To unsubscribe from this group, send email to in-portal-bug...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/in-portal-bugs?hl=en.
field_validation_min_len_fix.patch
Reply all
Reply to author
Forward
0 new messages