Short array syntax should be used to follow latest standards.Patch incoming.

CommentFileSizeAuthor
#3 short array 7-5.png900.19 KBriddhi.addweb
#2 multiple_registration_short_array_2969559_2.patch13.62 KBnkoporec

Comments

nkoporec created an issue. See original summary.

nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
commented
Assigned: nkoporec » Unassigned
Status: Active » Needs review
StatusFileSize
new multiple_registration_short_array_2969559_2.patch13.62 KB
riddhi.addweb’s picture

riddhi.addweb commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new short array 7-5.png900.19 KB

@nkoporec, Thanks for providing patch, your patch clears all the short array errors shown in pareview report. PFA

IreneV’s picture

IreneV commented

Works great

ysamoylenko’s picture

ysamoylenko commented

Thank you for the report and review. The patch will be included in the next module release.

  • ysamoylenko committed f41995d on 8.x-1.x authored by nkoporec 
    Issue #2969559 by nkoporec, riddhi.addweb, IreneV, ysamoylenko: Use...
ysamoylenko’s picture

ysamoylenko commented
Status: Reviewed & tested by the community » Fixed

The patch was committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.