Received: from malur.postgresql.org ([217.196.149.56]) by arkaria.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1hn2Nx-0003Le-EN for pgadmin-hackers@arkaria.postgresql.org; Mon, 15 Jul 2019 14:55:25 +0000 Received: from localhost ([127.0.0.1] helo=malur.postgresql.org) by malur.postgresql.org with esmtp (Exim 4.89) (envelope-from ) id 1hn2Nw-0001G6-A4 for pgadmin-hackers@arkaria.postgresql.org; Mon, 15 Jul 2019 14:55:24 +0000 Received: from magus.postgresql.org ([2a02:c0:301:0:ffff::29]) by malur.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1hn2Nv-0001Fz-Va for pgadmin-hackers@lists.postgresql.org; Mon, 15 Jul 2019 14:55:24 +0000 Received: from mail-wm1-x342.google.com ([2a00:1450:4864:20::342]) by magus.postgresql.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1hn2Nt-0005ZV-7v for pgadmin-hackers@postgresql.org; Mon, 15 Jul 2019 14:55:23 +0000 Received: by mail-wm1-x342.google.com with SMTP id s15so15545544wmj.3 for ; Mon, 15 Jul 2019 07:55:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=pgadmin.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=Simz09kCBOm0XuOc84Ty7eBcbcZY+NApc9iVPyPxolo=; b=Ae1SxVWahLmPdlHx0naERW9OYi/JP2bzeJiIyD+hn04jrLdAZb/bSCcUsF89I0K1id uLWFEro9ccOXN+WrYfOORPW4sDW+0/NqoRd0QHa46GJbsZ4kLOnCyfwEVgnmz874A/9Q I34QGJ18xBDY0Ba0+l/seLS/ryJcRa5NE9wStpZ/pt/O8fCBEzgOVCsd5k3I4Rc9uaag lMsdMrqWZhKksN/4Lm3dRDeSzO4hSfzvrQJqP9+teGkO2MjFHNosyZ8mpQb3qDyZxgB1 lriRDvvHVPbVba+VEJW4u9eLi+gROEaDr72Ri9tabOzq6Zv1lS5m6rR6Fp39/utPHq49 TRlQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=Simz09kCBOm0XuOc84Ty7eBcbcZY+NApc9iVPyPxolo=; b=PB/TER1nVctkQyCGRJot844+ZVU0TqMnoVP4MlbLBZw6kJPzIpj6yKwuzstO09ce4j 82jqUozrkuxVzzrk1vd1cavFWArheYszRDtMcSpA09G6ThgF9okFAOEWk08SKgYVeuxJ fOtGgzPrpqcKgsVQglwSfJqsEsma+OJVBd2eDTJuqhQOSxkDWYdNmSq0ZhQ8drqrc9qp remTnJcrrZrBXNCKg4Cu4KcD/Jd6jQAHt/mwtLCg1k35CTLRYTo4K9R0jRzad/aw+hmF 8XrAbIOgMhAYl3RswtFoLXnPBCCMG2Y1+D84gLiVSLN5+CjLtozlUJTNENPTyP1YLm5n e6NA== X-Gm-Message-State: APjAAAW0LQd9VEuXXcezjmUDwxJXsJL5jqD/WQdKTWvib3oO8O4q6HoB +jWxd1KTfUcuohcOiMiobZDJ/MG9kAfMV/cWQes4Uw== X-Google-Smtp-Source: APXvYqwWIiAYBN8QGoKht4nEmRnLtfAWERiyaJsdtZiF/K1eZTWDNs19dimn9trOqVr0YLUxA3VC4eYeQdNo2KFRuLU= X-Received: by 2002:a1c:f018:: with SMTP id a24mr24371315wmb.66.1563202518734; Mon, 15 Jul 2019 07:55:18 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: Dave Page Date: Mon, 15 Jul 2019 15:55:06 +0100 Message-ID: Subject: Re: [pgAdmin4][Patch]: RM 4428 and 4448 To: Akshay Joshi Cc: pgadmin-hackers Content-Type: multipart/alternative; boundary="000000000000d7729d058db972e8" List-Id: List-Help: List-Subscribe: List-Post: List-Owner: List-Archive: Precedence: bulk --000000000000d7729d058db972e8 Content-Type: text/plain; charset="UTF-8" Thanks, applied. On Mon, Jul 15, 2019 at 1:32 PM Akshay Joshi wrote: > Hi Hackers, > > Attached is the patch to fix following issues in pgagent module: > > - RM 4428 "malformed array literal error when updating pgagent job". > - RM 4448 "Error when updating connection string in pgagent Jobs." > - When user create a schedule using Create->Schedule dialog browser > tree is not showing newly created node. > - Properties tab showing same properties for all the created schedule. > - Added validation in "pga_jobstep.js", throws error on browser when > we modify step from the pgagent dialog and select the same node. > > > Added regression test cases to add schedule and step from the pgagent job > itself. > Please review it. > > -- > *Thanks & Regards* > *Akshay Joshi* > > *Sr. Software Architect* > *EnterpriseDB Software India Private Limited* > *Mobile: +91 976-788-8246* > -- Dave Page Blog: http://pgsnake.blogspot.com Twitter: @pgsnake EnterpriseDB UK: http://www.enterprisedb.com The Enterprise PostgreSQL Company --000000000000d7729d058db972e8 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Thanks, applied.

On Mon, Jul 15, 2019 at 1:32 PM Akshay Jos= hi <akshay.joshi@enterp= risedb.com> wrote:
Hi Hackers,

Attached is the = patch to fix following issues in pgagent module:
  • RM 4428 = "malformed array literal error when updating pgagent job".
  • RM 4448 "Error when updating connection string in pgagent Jobs."= ;
  • When user create a schedule using Create->Schedule dialog brow= ser tree is not showing newly created node.
  • Properties tab showing = same properties for all the created schedule.
  • Added validation in &= quot;pga_jobstep.js", throws error on browser when we modify step from= the pgagent dialog and select the same node.

Added regression test cases to add schedule and step from the pgagent= job itself.
Please review it.

--
Tha= nks & Regards
Akshay Joshi
Sr. Software Architect
EnterpriseDB Software India Pr= ivate Limited
Mobile: +91 976-788-8246


--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @p= gsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL = Company
--000000000000d7729d058db972e8--