Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: max_positive_value for Integer types #17712

Merged
merged 2 commits into from Aug 2, 2022
Merged

Conversation

s-aga-r
Copy link
Contributor

@s-aga-r s-aga-r commented Aug 2, 2022

Source / Reference: ISS-22-23-01561

The max values for Integer types are:

  • SMALLINT: (2**15 -1) = 32767
  • INT: (2**31 -1) = 2147483647
  • BIGINT: (2**63 -1) = 9223372036854775807

Before:

  • An error will be thrown if we try to set 2147483648 for INT. Ideally, it should be truncated.
Screen.Recording.2022-08-02.at.3.23.25.PM.mov

After:

Screen.Recording.2022-08-02.at.3.38.02.PM.mov

@s-aga-r s-aga-r requested a review from ankush August 2, 2022 10:12
@github-actions github-actions bot added the add-test-cases Add test case to validate fix or enhancement label Aug 2, 2022
@ankush ankush removed the add-test-cases Add test case to validate fix or enhancement label Aug 2, 2022
@ankush ankush added Skip CI Doesn't run Ci for this PR. backport version-13-hotfix backport version-14-hotfix backport to version 14 labels Aug 2, 2022
@ankush ankush merged commit d88d9f5 into frappe:develop Aug 2, 2022
mergify bot pushed a commit that referenced this pull request Aug 2, 2022
* fix: max_positive_value for Integer types

* style: formatting

Co-authored-by: Ankush Menat <ankushmenat@gmail.com>
(cherry picked from commit d88d9f5)
mergify bot pushed a commit that referenced this pull request Aug 2, 2022
* fix: max_positive_value for Integer types

* style: formatting

Co-authored-by: Ankush Menat <ankushmenat@gmail.com>
(cherry picked from commit d88d9f5)
ankush pushed a commit that referenced this pull request Aug 2, 2022
* fix: max_positive_value for Integer types

* style: formatting

Co-authored-by: Ankush Menat <ankushmenat@gmail.com>
(cherry picked from commit d88d9f5)

Co-authored-by: Sagar Sharma <sagarsharma.s312@gmail.com>
ankush pushed a commit that referenced this pull request Aug 2, 2022
* fix: max_positive_value for Integer types

* style: formatting

Co-authored-by: Ankush Menat <ankushmenat@gmail.com>
(cherry picked from commit d88d9f5)

Co-authored-by: Sagar Sharma <sagarsharma.s312@gmail.com>
frappe-pr-bot pushed a commit that referenced this pull request Aug 3, 2022
## [14.0.1](v14.0.0...v14.0.1) (2022-08-03)

### Bug Fixes

* Append to condition misleading message ([#17696](#17696)) ([#17709](#17709)) ([f05b517](f05b517))
* **global_search:** Trigger rebuilding on Custom Field's property change ([#17706](#17706)) ([a5d576f](a5d576f))
* max_positive_value for Integer types ([#17712](#17712)) ([#17714](#17714)) ([306af63](306af63))
* set /app as redirect for pageview home button ([#17715](#17715)) ([#17717](#17717)) ([594944a](594944a))
* translate each role ([6816f83](6816f83))
* use `throw` ([f7ce174](f7ce174))
* use warn util ([c66fa28](c66fa28))

### Performance Improvements

* reduce DB call in `frappe.client.get` ([#17665](#17665)) ([#17711](#17711)) ([1655fb3](1655fb3))
* specify reference doctype in filters ([#17704](#17704)) ([fc230b4](fc230b4))
developmentforpeople pushed a commit to developmentforpeople/frappe that referenced this pull request Aug 7, 2022
## [14.0.1](frappe/frappe@v14.0.0...v14.0.1) (2022-08-03)

### Bug Fixes

* Append to condition misleading message ([frappe#17696](frappe#17696)) ([frappe#17709](frappe#17709)) ([f05b517](frappe@f05b517))
* **global_search:** Trigger rebuilding on Custom Field's property change ([frappe#17706](frappe#17706)) ([a5d576f](frappe@a5d576f))
* max_positive_value for Integer types ([frappe#17712](frappe#17712)) ([frappe#17714](frappe#17714)) ([306af63](frappe@306af63))
* set /app as redirect for pageview home button ([frappe#17715](frappe#17715)) ([frappe#17717](frappe#17717)) ([594944a](frappe@594944a))
* translate each role ([6816f83](frappe@6816f83))
* use `throw` ([f7ce174](frappe@f7ce174))
* use warn util ([c66fa28](frappe@c66fa28))

### Performance Improvements

* reduce DB call in `frappe.client.get` ([frappe#17665](frappe#17665)) ([frappe#17711](frappe#17711)) ([1655fb3](frappe@1655fb3))
* specify reference doctype in filters ([frappe#17704](frappe#17704)) ([fc230b4](frappe@fc230b4))
frappe-pr-bot pushed a commit that referenced this pull request Aug 9, 2022
# [13.37.0](v13.36.3...v13.37.0) (2022-08-09)

### Bug Fixes

* allow system managers to toggle email queue ([c004846](c004846))
* allow to import time field ([#17677](#17677)) ([#17690](#17690)) ([59f58c7](59f58c7))
* check permission before sending email ([bb73014](bb73014))
* Dropdown selection list in Reports should be translatable ([#17679](#17679)) ([8c0e5ab](8c0e5ab))
* **global_search:** Trigger rebuilding on Custom Field's property change ([#17705](#17705)) ([b4ff5b2](b4ff5b2))
* max_positive_value for Integer types ([#17712](#17712)) ([#17713](#17713)) ([e173e64](e173e64))
* merge conflict ([2f71fb1](2f71fb1))
* restored patch and remove unused import ([b2232f1](b2232f1))
* return promise ([#17646](#17646)) ([#17666](#17666)) ([d931e26](d931e26))
* Show Report & Dashboard View for File Doctype (backport [#17688](#17688)) ([#17695](#17695)) ([ad09e87](ad09e87))
* version bump regex in automated release ([#17725](#17725)) ([#17726](#17726)) ([a4f682a](a4f682a))

### Features

* Role based permission for Dashboard Chart ([#17634](#17634)) ([#17655](#17655)) ([c3f45e7](c3f45e7))

### Performance Improvements

* reduce DB calls in `frappe.client.get` (and other changes) (backport [#17665](#17665)) ([#17710](#17710)) ([5fe4924](5fe4924))
* reduce DB calls made in `get_fetch_values` ([#17671](#17671)) ([#17738](#17738)) ([fc58a87](fc58a87))
* specify reference doctype in filters ([#17703](#17703)) ([88a59d8](88a59d8))
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 5, 2022
@s-aga-r s-aga-r added bug and removed backport version-13-hotfix Skip CI Doesn't run Ci for this PR. backport version-14-hotfix backport to version 14 labels Sep 21, 2022
@s-aga-r s-aga-r deleted the fix/max-value branch November 12, 2022 12:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants