| From: | Harshal Dhumal <harshal(dot)dhumal(at)enterprisedb(dot)com> |
|---|---|
| To: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
| Subject: | variable scope issue in server js [pgadmin4] |
| Date: | 2016-05-12 11:35:51 |
| Message-ID: | CAFiP3vyJzWaBBC9e+bVP29idccyojjOqwwcyzz9Uh49SQJTUNQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgadmin-hackers |
Hi,
PFA patch for server.js. Replaced ";" (semicolon) with "," (coma) in
variable declaration.
Issue: This was changing scope of all the variables declared after ";"
(semicolon) from *local* to *global* and therefore affecting all other *local
variables *with same name at different place in same file.
--
*Harshal Dhumal*
*Software Engineer *
EenterpriseDB <http://www.enterprisedb.com>
| Attachment | Content-Type | Size |
|---|---|---|
| server_js_variable_scope_issue.patch | text/x-patch | 2.8 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Murtuza Zabuawala | 2016-05-12 11:57:07 | Re: PATCH: Added "Named restore point" functionality (pgAdmin4) |
| Previous Message | Akshay Joshi | 2016-05-12 11:15:44 | Re: PATCH: Added "Named restore point" functionality (pgAdmin4) |