database-query() has been used at several places which should not be used.
database->select() should be used instead of database->query().

CommentFileSizeAuthor
#5 2952479-5-database-select-query-used.patch2.15 KBanmolgoyal74
#3 2952479-3-database-select-query-used.patch2.5 KBanmolgoyal74
#2 2952479-2-database-select-query-used.patch2.96 KBanmolgoyal74

Comments

anmolgoyal74 created an issue. See original summary.

anmolgoyal74’s picture

anmolgoyal74 commented
Status: Active » Needs review
StatusFileSize
new 2952479-2-database-select-query-used.patch2.96 KB

This patch will solve the issue.

anmolgoyal74’s picture

anmolgoyal74 commented
StatusFileSize
new 2952479-3-database-select-query-used.patch2.5 KB

Please use this patch instead of patch given in comment 2.

nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
commented
Status: Needs review » Needs work

A couple of warnings when I try to apply the patch :

query2.patch:25: trailing whitespace.
    }  
query2.patch:41: new blank line at EOF.
+
warning: 2 lines add whitespace errors.
anmolgoyal74’s picture

anmolgoyal74 commented
Status: Needs work » Needs review
StatusFileSize
new 2952479-5-database-select-query-used.patch2.15 KB
nkoporec’s picture

nkoporec
Primary language English
Location Slovenia
commented
Status: Needs review » Reviewed & tested by the community

Hi @AnmolgoyalL74, tested your latest patch and now it applies cleanly.Good job!

gaurav.kapoor’s picture

gaurav.kapoor commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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