This warning is displayed on the Node add/edit form, when there is a multifield on the form

Patch attached

CommentFileSizeAuthor
#6 multifield-2647990-6.patch862 bytesbdone
#6 interdiff-2647990-3-6.txt1011 bytesbdone
#6 multifield-2647990-6.patch862 bytesbdone
#5 interdiff-2647990-3-5.txt1010 bytesbdone
#5 multifield-2647990-5.patch861 bytesbdone
#3 interdiff-2647990-1-3.txt796 bytesheykarthikwithu
#3 2647990-3.patch845 bytesheykarthikwithu
multifield-max_delta-warning.patch848 bytescslevy

Comments

cslevy created an issue. See original summary.

cslevy’s picture

cslevy commented
Status: Active » Needs review
heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
StatusFileSize
new 2647990-3.patch845 bytes
new interdiff-2647990-1-3.txt796 bytes

Checked with multifield-7.x-1.x version, but was not able to reproduce the same.

Even though the patch will be ok for checking the value is existing or not, since already piece of condition is written like > 0 for checking the #maxdelta.
So for that isset() will be the best choice for checking the empty condition, since #maxdelta this is an integer value.

heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
Issue tags: +Quick fix
bdone’s picture

bdone commented
StatusFileSize
new multifield-2647990-5.patch861 bytes
new interdiff-2647990-3-5.txt1010 bytes

re-roll for 7.x-1.x-dev.

bdone’s picture

bdone commented
StatusFileSize
new multifield-2647990-6.patch862 bytes
new interdiff-2647990-3-6.txt1011 bytes
new multifield-2647990-6.patch862 bytes