Bug #53818 [Asn->Csd]: Missing info in pg_get_notify function

2012-01-11 Thread r dot i dot k at free dot fr
Edit report at https://bugs.php.net/bug.php?id=53818&edit=1

 ID: 53818
 User updated by:    r dot i dot k at free dot fr
 Reported by:    r dot i dot k at free dot fr
 Summary:Missing info in pg_get_notify function
-Status: Assigned
+Status: Closed
 Type:   Bug
 Package:PostgreSQL related
 Operating System:   Linux
 PHP Version:5.3.8
 Assigned To:iliaa
 Block user comment: N
 Private report: N

 New Comment:

I confirm that the current patch has been applied to PHP 5.3.9
Just disappointed to have waited for quite a year, for such a full working 
patch integration.

This situation does not tempt me to contribute anymore... 

Bug CLOSED.


Previous Comments:

[2012-01-11 08:50:47] timosha at gmail dot com

hmm, seems that this patch applied for php-5.3.9 but no info in changelog


[2011-10-15 16:19:12] r dot i dot k at free dot fr

The provided patch, 9 months ago, is still missing from 5.3 branch...
It would be cool to have it pushed in official source code, since it still 
works (made for 5.2.17, applyable to 5.3.8), and give access to a feature 
appeared in PostgreSQL 14 months ago :(((


[2011-10-14 06:06:49] timosha at gmail dot com

Please backport this patch to 5.3.x. :)


[2011-01-27 17:02:36] il...@php.net

This bug has been fixed in SVN.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.




[2011-01-22 19:32:25] r dot i dot k at free dot fr

Description:

Since Postgresql 9.0, event notifications fired by the NOTIFY sql command 
includes a payload (a 8000 bytes max piece of text).

As of now, the PHP function pg_get_notify only retrieve the "channel" (name of 
the notification) and the backend pid.

The attached patch improves this situation by adding the payload to the 
returned array of the pg_get_notify function, when the connected PG backend 
version is at least 9.0.

Hope it helps...

Test script:
---
$dbconn = pg_connect(...) ; // To be completed

pg_query($dbconn, "LISTEN toto");
pg_query($dbconn, "NOTIFY toto, 'Hello world !'");

$notify = pg_get_notify($dbconn, PGSQL_ASSOC) ;

print_r($notify) ;


Expected result:

// Array("message" => "toto", "pid" => 12345, "payload" => "Hello world !")

Actual result:
--
// Array("message" => "toto", "pid" => 12345)






-- 
Edit this bug report at https://bugs.php.net/bug.php?id=53818&edit=1


Bug #53818 [Csd->Asn]: Missing info in pg_get_notify function

2011-10-15 Thread r dot i dot k at free dot fr
Edit report at https://bugs.php.net/bug.php?id=53818&edit=1

 ID: 53818
 User updated by:    r dot i dot k at free dot fr
 Reported by:    r dot i dot k at free dot fr
 Summary:Missing info in pg_get_notify function
-Status: Closed
+Status: Assigned
 Type:   Bug
 Package:PostgreSQL related
 Operating System:   Linux
-PHP Version:5.2.17
+PHP Version:5.3.8
 Assigned To:iliaa
 Block user comment: N
 Private report: N

 New Comment:

The provided patch, 9 months ago, is still missing from 5.3 branch...
It would be cool to have it pushed in official source code, since it still 
works (made for 5.2.17, applyable to 5.3.8), and give access to a feature 
appeared in PostgreSQL 14 months ago :(((


Previous Comments:

[2011-10-14 06:06:49] timosha at gmail dot com

Please backport this patch to 5.3.x. :)


[2011-01-27 17:02:36] il...@php.net

This bug has been fixed in SVN.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
 
Thank you for the report, and for helping us make PHP better.




[2011-01-22 19:32:25] r dot i dot k at free dot fr

Description:

Since Postgresql 9.0, event notifications fired by the NOTIFY sql command 
includes a payload (a 8000 bytes max piece of text).

As of now, the PHP function pg_get_notify only retrieve the "channel" (name of 
the notification) and the backend pid.

The attached patch improves this situation by adding the payload to the 
returned array of the pg_get_notify function, when the connected PG backend 
version is at least 9.0.

Hope it helps...

Test script:
---
$dbconn = pg_connect(...) ; // To be completed

pg_query($dbconn, "LISTEN toto");
pg_query($dbconn, "NOTIFY toto, 'Hello world !'");

$notify = pg_get_notify($dbconn, PGSQL_ASSOC) ;

print_r($notify) ;


Expected result:

// Array("message" => "toto", "pid" => 12345, "payload" => "Hello world !")

Actual result:
--
// Array("message" => "toto", "pid" => 12345)






-- 
Edit this bug report at https://bugs.php.net/bug.php?id=53818&edit=1


[PHP-BUG] Bug #53818 [NEW]: Missing info in pg_get_notify function

2011-01-22 Thread r dot i dot k at free dot fr
From: 
Operating system: Linux
PHP version:  5.2.17
Package:  PostgreSQL related
Bug Type: Bug
Bug description:Missing info in pg_get_notify function

Description:

Since Postgresql 9.0, event notifications fired by the NOTIFY sql command
includes a payload (a 8000 bytes max piece of text).



As of now, the PHP function pg_get_notify only retrieve the "channel" (name
of the notification) and the backend pid.



The attached patch improves this situation by adding the payload to the
returned array of the pg_get_notify function, when the connected PG backend
version is at least 9.0.



Hope it helps...

Test script:
---
$dbconn = pg_connect(...) ; // To be completed



pg_query($dbconn, "LISTEN toto");

pg_query($dbconn, "NOTIFY toto, 'Hello world !'");



$notify = pg_get_notify($dbconn, PGSQL_ASSOC) ;



print_r($notify) ;



Expected result:

// Array("message" => "toto", "pid" => 12345, "payload" => "Hello world !")

Actual result:
--
// Array("message" => "toto", "pid" => 12345)

-- 
Edit bug report at http://bugs.php.net/bug.php?id=53818&edit=1
-- 
Try a snapshot (PHP 5.2):
http://bugs.php.net/fix.php?id=53818&r=trysnapshot52
Try a snapshot (PHP 5.3):
http://bugs.php.net/fix.php?id=53818&r=trysnapshot53
Try a snapshot (trunk):  
http://bugs.php.net/fix.php?id=53818&r=trysnapshottrunk
Fixed in SVN:
http://bugs.php.net/fix.php?id=53818&r=fixed
Fixed in SVN and need be documented: 
http://bugs.php.net/fix.php?id=53818&r=needdocs
Fixed in release:
http://bugs.php.net/fix.php?id=53818&r=alreadyfixed
Need backtrace:  
http://bugs.php.net/fix.php?id=53818&r=needtrace
Need Reproduce Script:   
http://bugs.php.net/fix.php?id=53818&r=needscript
Try newer version:   
http://bugs.php.net/fix.php?id=53818&r=oldversion
Not developer issue: 
http://bugs.php.net/fix.php?id=53818&r=support
Expected behavior:   
http://bugs.php.net/fix.php?id=53818&r=notwrong
Not enough info: 
http://bugs.php.net/fix.php?id=53818&r=notenoughinfo
Submitted twice: 
http://bugs.php.net/fix.php?id=53818&r=submittedtwice
register_globals:
http://bugs.php.net/fix.php?id=53818&r=globals
PHP 4 support discontinued:  http://bugs.php.net/fix.php?id=53818&r=php4
Daylight Savings:http://bugs.php.net/fix.php?id=53818&r=dst
IIS Stability:   
http://bugs.php.net/fix.php?id=53818&r=isapi
Install GNU Sed: 
http://bugs.php.net/fix.php?id=53818&r=gnused
Floating point limitations:  
http://bugs.php.net/fix.php?id=53818&r=float
No Zend Extensions:  
http://bugs.php.net/fix.php?id=53818&r=nozend
MySQL Configuration Error:   
http://bugs.php.net/fix.php?id=53818&r=mysqlcfg