public inbox for [email protected]  
help / color / mirror / Atom feed
PATCH: Comments are not visible in Language node (pgAdmin4)
2+ messages / 2 participants
[nested] [flat]

* PATCH: Comments are not visible in Language node (pgAdmin4)
@ 2016-07-12 11:15  Murtuza Zabuawala <[email protected]>
  0 siblings, 1 reply; 2+ messages in thread

From: Murtuza Zabuawala @ 2016-07-12 11:15 UTC (permalink / raw)
  To: pgadmin-hackers

Hi,

PFA patch to fix RM#1412, where comments are not visible in sql pane for
Languages node.

--
Regards,
Murtuza Zabuawala
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


-- 
Sent via pgadmin-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers


Attachments:

  [application/octet-stream] RM_1421.patch (2.4K, 3-RM_1421.patch)
  download | inline diff:
diff --git a/web/pgadmin/browser/server_groups/servers/databases/languages/templates/languages/sql/9.1_plus/sqlpane.sql b/web/pgadmin/browser/server_groups/servers/databases/languages/templates/languages/sql/9.1_plus/sqlpane.sql
index a7e51a9..f6978ba 100644
--- a/web/pgadmin/browser/server_groups/servers/databases/languages/templates/languages/sql/9.1_plus/sqlpane.sql
+++ b/web/pgadmin/browser/server_groups/servers/databases/languages/templates/languages/sql/9.1_plus/sqlpane.sql
@@ -5,16 +5,22 @@
 {# ============= CREATE LANGUAGE Query ============= #}
 CREATE {% if data.trusted %}TRUSTED{% endif %} PROCEDURAL LANGUAGE {{ conn|qtIdent(data.name) }}
 {% if data.lanproc %}
-  HANDLER {{ conn|qtIdent(data.lanproc) }}
+    HANDLER {{ conn|qtIdent(data.lanproc) }}
 {% endif %}
 {% if data.laninl %}
-  INLINE {{ conn|qtIdent(data.laninl) }}
+    INLINE {{ conn|qtIdent(data.laninl) }}
 {% endif %}
 {% if data.lanval %}
-  VALIDATOR {{ conn|qtIdent(data.lanval) }}{% endif %};
+    VALIDATOR {{ conn|qtIdent(data.lanval) }}{% endif %};
   
 {# ============= ALTER LANGUAGE Query ============= #}
 {% if data.lanowner %}
 ALTER LANGUAGE {{ conn|qtIdent(data.name) }}
-  OWNER TO {{ conn|qtIdent(data.lanowner) }};
+    OWNER TO {{ conn|qtIdent(data.lanowner) }};
+{% endif %}
+
+{# ============= Comment on LANGUAGE Query ============= #}
+{% if data.description %}
+COMMENT ON LANGUAGE {{ conn|qtIdent(data.name) }}
+    IS {{ data.description|qtLiteral }};
 {% endif %}
\ No newline at end of file
diff --git a/web/pgadmin/browser/server_groups/servers/databases/languages/templates/languages/sql/9.3_plus/sqlpane.sql b/web/pgadmin/browser/server_groups/servers/databases/languages/templates/languages/sql/9.3_plus/sqlpane.sql
index d8a3d36..905ef1a 100644
--- a/web/pgadmin/browser/server_groups/servers/databases/languages/templates/languages/sql/9.3_plus/sqlpane.sql
+++ b/web/pgadmin/browser/server_groups/servers/databases/languages/templates/languages/sql/9.3_plus/sqlpane.sql
@@ -17,4 +17,10 @@ CREATE {% if data.trusted %}TRUSTED{% endif %} PROCEDURAL LANGUAGE {{ conn|qtIde
 {% if data.lanowner %}
 ALTER LANGUAGE {{ conn|qtIdent(data.name) }}
     OWNER TO {{ conn|qtIdent(data.lanowner) }};
+{% endif %}
+
+{# ============= Comment on LANGUAGE Query ============= #}
+{% if data.description %}
+COMMENT ON LANGUAGE {{ conn|qtIdent(data.name) }}
+    IS {{ data.description|qtLiteral }};
 {% endif %}
\ No newline at end of file


^ permalink  raw  reply  [nested|flat] 2+ messages in thread

* Re: PATCH: Comments are not visible in Language node (pgAdmin4)
@ 2016-07-13 13:45  Dave Page <[email protected]>
  parent: Murtuza Zabuawala <[email protected]>
  0 siblings, 0 replies; 2+ messages in thread

From: Dave Page @ 2016-07-13 13:45 UTC (permalink / raw)
  To: Murtuza Zabuawala <[email protected]>; +Cc: pgadmin-hackers

Thanks, applied.

On Tue, Jul 12, 2016 at 12:15 PM, Murtuza Zabuawala
<[email protected]> wrote:
> Hi,
>
> PFA patch to fix RM#1412, where comments are not visible in sql pane for
> Languages node.
>
> --
> Regards,
> Murtuza Zabuawala
> EnterpriseDB: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
>
> --
> Sent via pgadmin-hackers mailing list ([email protected])
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgadmin-hackers
>



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

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


-- 
Sent via pgadmin-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgadmin-hackers




^ permalink  raw  reply  [nested|flat] 2+ messages in thread


end of thread, other threads:[~2016-07-13 13:45 UTC | newest]

Thread overview: 2+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2016-07-12 11:15 PATCH: Comments are not visible in Language node (pgAdmin4) Murtuza Zabuawala <[email protected]>
2016-07-13 13:45 ` Dave Page <[email protected]>

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox