Problem/Motivation

Arrays in the module are not updated with short array syntax.

Proposed resolution

Convert all the arrays to short array syntax.

Remaining tasks

N/A

User interface changes

N/A

API changes

N/A

Data model changes

N/A

CommentFileSizeAuthor
#5 coding-2931656-5.patch2.4 KBpvsureshmca
#3 FZ152_File.png36 KBriddhi.addweb
#2 coding-2931656-2.patch1.64 KBpvsureshmca

Comments

pvsureshmca created an issue. See original summary.

pvsureshmca’s picture

pvsureshmca commented
Assigned: pvsureshmca » Unassigned
Status: Needs work » Needs review
StatusFileSize
new coding-2931656-2.patch1.64 KB

Hi

I have added patch for fixing new array coding standard in the module file.

riddhi.addweb’s picture

riddhi.addweb commented
Status: Needs review » Needs work
StatusFileSize
new FZ152_File.png36 KB

@pvsureshmca, Thanks for providing patch & solving it. Your patch corrects the short array errors, but I found that in file it is missing.
PFA in which i mentioned missing file.

pvsureshmca’s picture

pvsureshmca commented
Assigned: Unassigned » pvsureshmca
pvsureshmca’s picture

pvsureshmca commented
Status: Needs work » Needs review
StatusFileSize
new coding-2931656-5.patch2.4 KB

@Jigar.addweb,

Thanks for the review and shared your feedback. I have added one more patch for both files.

Please review it.

Thanks again.

pvsureshmca’s picture

pvsureshmca commented
Assigned: pvsureshmca » Unassigned
riddhi.addweb’s picture

riddhi.addweb commented
Status: Needs review » Reviewed & tested by the community

@pvsureshmca Thanks for quick reply and correction. I have checked your latest patch and now all file changes done and looks good to me.

adubovskoy’s picture

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

Status: Fixed » Closed (fixed)

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