Skip to content

Commit

Permalink
Merge pull request #8 from tdesvenain/master
Browse files Browse the repository at this point in the history
Fix description of min max field: actually, max value is included, not excluded
  • Loading branch information
mgedmin committed Apr 18, 2014
2 parents 564acd4 + 3c202b4 commit 978f020
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions CHANGES.rst
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@ zope.schema Changelog
4.4.2 (unreleased)
------------------

- Fix description of min max field: actually, max value is included, not excluded.

- TBD


Expand Down
2 changes: 1 addition & 1 deletion src/zope/schema/interfaces.py
Original file line number Diff line number Diff line change
Expand Up @@ -363,7 +363,7 @@ class IInt(IMinMax, IField):
)

max = Int(
title=_("End of the range (excluding the value itself)"),
title=_("End of the range (including the value itself)"),
required=False,
default=None
)
Expand Down

3 comments on commit 978f020

@tdesvenain
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, can we have a release for this please ?

@tseaver
Copy link
Member

@tseaver tseaver commented on 978f020 Sep 4, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tdesvenain
Copy link
Contributor

@tdesvenain tdesvenain commented on 978f020 Sep 4, 2014 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.