On Thu, Jul 27, 2017 at 2:41 PM, Akshay Joshi <akshay.jo...@enterprisedb.com > wrote:
> Hi All > > As in commit "Update alertify alerts to use the styling defined in > the style guide": > > https://git.postgresql.org/gitweb/?p=pgadmin4.git;a=commitdiff;h= > 2a30a86e7d5e562040500f448fbb0d143ff2cff9 > > https://git.postgresql.org/gitweb/?p=pgadmin4.git;a=commitdiff;h= > f2d2075d81718ec02550fb592851aa330d327b24 > > We have introduce new wrapper class "AlertifyWrapper" and replace calls > to alertify.success and alertify.error with following two lines in most > of the files > > var alertifyWrapper = new AlertifyWrapper(); > > alertifyWrapper.success(message); or alertifyWrapper.error(message); > > For each call we are creating dynamic object of AlertifyWrapper and call > the appropriate function. For example there are 20 such calls in a single > js file every time are are creating object and call appropriate function. > > > I have tried to improve the logic here and implemented it as below: > > - Extend alertify and move success, error and info functions from " > alertify_wrapper.js" file to "alertify.pgadmin.defaults.js", there > will be no use of "alertify_wrapper.js" > - Modify only "server.js" as POC, remove 'alertify' and replace > 'sources/alerts/alertify_wrapper' with 'pgadmin.alertifyjs' which is > nothing but mapping of "alertify.pgadmin.defaults.js" from defines and > named the reference object to 'alertify' so no need to change any function > call like "alertify.success, alertify.error". > > One more benefit of the above approach is if in future we want to use the > same style for alertify.warning, alertify.info, alertify.message etc.., > we will just have to extend that method in "alertify.pgadmin.defaults.js" > and no need to change the rest of the function call with AlertifyWrapper. > > > Attached is the POC patch, if it looks good then I'll start working on > replacing AlertifyWrapper with the above mentioned approach. > I like the approach - it's definitely cleaner, and saves instantiating a new object every time. -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company