FAQ
Edit report at http://pear.php.net/bugs/bug.php?id=18292&edit=1

ID: 18292
Updated by: till@php.net
Reported By: hidenobu dot shinozuka at incs dot co dot jp
Summary: S3 and Stream have a different option name.
-Status: Open
+Status: Bogus
Type: Bug
Package: Services_Amazon_S3
Operating System: Centos 5.5
Package Version: 0.3.1
PHP Version: 5.3.0
Roadmap Versions:
New Comment:

-Status: Open
+Status: Bogus
Sorry, but your problem does not imply a bug in PEAR itself. For a
list of more appropriate places to ask for help using PEAR, please
visit http://pear.php.net/support/ as this bug system is not the
appropriate forum for asking support questions.

Thank you for your interest in PEAR.

One is an option, one is a class variable. Class variables follow
camelcase, as do class methods.

All other config options use an underscore, I don't think this is
inconsistent - it would be if it was otherwise. If we change
one option, we'll have to change all. Add to this, this package is
actually used in many places and despite the lower
version number I'm against breaking BC.


Previous Comments:
------------------------------------------------------------------------

[2011-02-22 09:20:39] hiderer

Description:
------------
Http_option is defined "$httpConfig" at S3.php line 162, But in
Stream.php specifying "httpOptions" or "http_options" at line 940
to 944.

"httpConfig" unified hope.

Test script:
---------------
Services_Amazon_S3_Stream::register('s3',
array(
'access_key_id' => $access_key,
'secret_access_key' => $secret_key,
'http_config' => array('proxy_host' => '172.24.xx.xx', 'proxy_port' =>
8080)
));

$streamS3 = fopen('s3://mybucket/mykey', 'w');

------------------------------------------------------------------------

Search Discussions

  • Mike at Mar 28, 2011 at 2:04 pm
    Edit report at http://pear.php.net/bugs/bug.php?id=18292&edit=1

    ID: 18292
    Updated by: mike@silverorange.com
    Reported By: hidenobu dot shinozuka at incs dot co dot jp
    Summary: S3 and Stream have a different option name.
    -Status: Bogus
    +Status: Assigned
    Type: Bug
    Package: Services_Amazon_S3
    -Operating System: Centos 5.5
    +Operating System: Irrelevant
    Package Version: 0.3.1
    -PHP Version: 5.3.0
    +PHP Version: Irrelevant
    Roadmap Versions:
    New Comment:

    -Status: Bogus
    +Status: Assigned
    -Operating System: Centos 5.5
    +Operating System: Irrelevant
    -PHP Version: 5.3.0
    +PHP Version: Irrelevant
    I looked over the code and the original reporter is correct. The
    variable name used in
    Stream.php is wrong. It's a bug, not a preference.

    As for the config name, I think we should continue to support
    'http_option' for
    backwards-compatibility, but also add 'http_config' as the word 'config'
    is used
    throughout HTTP_Request2.


    Previous Comments:
    ------------------------------------------------------------------------

    [2011-03-27 16:07:05] till

    -Status: Open
    +Status: Bogus
    Sorry, but your problem does not imply a bug in PEAR itself. For a
    list of more appropriate places to ask for help using PEAR, please
    visit http://pear.php.net/support/ as this bug system is not the
    appropriate forum for asking support questions.

    Thank you for your interest in PEAR.

    One is an option, one is a class variable. Class variables follow
    camelcase, as do class methods.

    All other config options use an underscore, I don't think this is
    inconsistent - it would be if it was otherwise. If we change
    one option, we'll have to change all. Add to this, this package is
    actually used in many places and despite the lower
    version number I'm against breaking BC.

    ------------------------------------------------------------------------

    [2011-02-22 09:20:39] hiderer

    Description:
    ------------
    Http_option is defined "$httpConfig" at S3.php line 162, But in
    Stream.php specifying "httpOptions" or "http_options" at line 940
    to 944.

    "httpConfig" unified hope.

    Test script:
    ---------------
    Services_Amazon_S3_Stream::register('s3',
    array(
    'access_key_id' => $access_key,
    'secret_access_key' => $secret_key,
    'http_config' => array('proxy_host' => '172.24.xx.xx', 'proxy_port' =>
    8080)
    ));

    $streamS3 = fopen('s3://mybucket/mykey', 'w');

    ------------------------------------------------------------------------
  • Mike at Mar 28, 2011 at 2:05 pm
    Edit report at http://pear.php.net/bugs/bug.php?id=18292&edit=1

    ID: 18292
    Updated by: mike@silverorange.com
    Reported By: hidenobu dot shinozuka at incs dot co dot jp
    Summary: S3 and Stream have a different option name.
    Status: Open
    Type: Bug
    Package: Services_Amazon_S3
    Operating System: Irrelevant
    Package Version: 0.3.1
    PHP Version: Irrelevant
    -Assigned To:
    +Assigned To: gauthierm
    Roadmap Versions:
    New Comment:

    -Assigned To:
    +Assigned To: gauthierm



    Previous Comments:
    ------------------------------------------------------------------------

    [2011-03-28 15:06:58] gauthierm

    -Status: Bogus
    +Status: Assigned
    -Operating System: Centos 5.5
    +Operating System: Irrelevant
    -PHP Version: 5.3.0
    +PHP Version: Irrelevant
    I looked over the code and the original reporter is correct. The
    variable name used in
    Stream.php is wrong. It's a bug, not a preference.

    As for the config name, I think we should continue to support
    'http_option' for
    backwards-compatibility, but also add 'http_config' as the word 'config'
    is used
    throughout HTTP_Request2.

    ------------------------------------------------------------------------

    [2011-03-27 16:07:05] till

    -Status: Open
    +Status: Bogus
    Sorry, but your problem does not imply a bug in PEAR itself. For a
    list of more appropriate places to ask for help using PEAR, please
    visit http://pear.php.net/support/ as this bug system is not the
    appropriate forum for asking support questions.

    Thank you for your interest in PEAR.

    One is an option, one is a class variable. Class variables follow
    camelcase, as do class methods.

    All other config options use an underscore, I don't think this is
    inconsistent - it would be if it was otherwise. If we change
    one option, we'll have to change all. Add to this, this package is
    actually used in many places and despite the lower
    version number I'm against breaking BC.

    ------------------------------------------------------------------------

    [2011-02-22 09:20:39] hiderer

    Description:
    ------------
    Http_option is defined "$httpConfig" at S3.php line 162, But in
    Stream.php specifying "httpOptions" or "http_options" at line 940
    to 944.

    "httpConfig" unified hope.

    Test script:
    ---------------
    Services_Amazon_S3_Stream::register('s3',
    array(
    'access_key_id' => $access_key,
    'secret_access_key' => $secret_key,
    'http_config' => array('proxy_host' => '172.24.xx.xx', 'proxy_port' =>
    8080)
    ));

    $streamS3 = fopen('s3://mybucket/mykey', 'w');

    ------------------------------------------------------------------------
  • Mike at Mar 28, 2011 at 2:33 pm
    Edit report at http://pear.php.net/bugs/bug.php?id=18292&edit=1

    ID: 18292
    Updated by: mike@silverorange.com
    Reported By: hidenobu dot shinozuka at incs dot co dot jp
    Summary: S3 and Stream have a different option name.
    -Status: Assigned
    +Status: Closed
    Type: Bug
    Package: Services_Amazon_S3
    Operating System: Irrelevant
    Package Version: 0.3.1
    PHP Version: Irrelevant
    Assigned To: gauthierm
    Roadmap Versions:
    New Comment:

    -Status: Assigned
    +Status: Closed
    This bug has been fixed in SVN.

    If this was a documentation problem, the fix will appear on pear.php.net
    by the end of next Sunday (CET).

    If this was a problem with the pear.php.net website, the change should
    be live shortly.

    Otherwise, the fix will appear in the package's next release.

    Thank you for the report and for helping us make PEAR better.

    Fixed in http://svn.php.net/viewvc?view=revision&revision=309787


    Previous Comments:
    ------------------------------------------------------------------------

    [2011-03-28 15:07:52] gauthierm

    -Assigned To:
    +Assigned To: gauthierm


    ------------------------------------------------------------------------

    [2011-03-28 15:06:58] gauthierm

    -Status: Bogus
    +Status: Assigned
    -Operating System: Centos 5.5
    +Operating System: Irrelevant
    -PHP Version: 5.3.0
    +PHP Version: Irrelevant
    I looked over the code and the original reporter is correct. The
    variable name used in
    Stream.php is wrong. It's a bug, not a preference.

    As for the config name, I think we should continue to support
    'http_option' for
    backwards-compatibility, but also add 'http_config' as the word 'config'
    is used
    throughout HTTP_Request2.

    ------------------------------------------------------------------------

    [2011-03-27 16:07:05] till

    -Status: Open
    +Status: Bogus
    Sorry, but your problem does not imply a bug in PEAR itself. For a
    list of more appropriate places to ask for help using PEAR, please
    visit http://pear.php.net/support/ as this bug system is not the
    appropriate forum for asking support questions.

    Thank you for your interest in PEAR.

    One is an option, one is a class variable. Class variables follow
    camelcase, as do class methods.

    All other config options use an underscore, I don't think this is
    inconsistent - it would be if it was otherwise. If we change
    one option, we'll have to change all. Add to this, this package is
    actually used in many places and despite the lower
    version number I'm against breaking BC.

    ------------------------------------------------------------------------

    [2011-02-22 09:20:39] hiderer

    Description:
    ------------
    Http_option is defined "$httpConfig" at S3.php line 162, But in
    Stream.php specifying "httpOptions" or "http_options" at line 940
    to 944.

    "httpConfig" unified hope.

    Test script:
    ---------------
    Services_Amazon_S3_Stream::register('s3',
    array(
    'access_key_id' => $access_key,
    'secret_access_key' => $secret_key,
    'http_config' => array('proxy_host' => '172.24.xx.xx', 'proxy_port' =>
    8080)
    ));

    $streamS3 = fopen('s3://mybucket/mykey', 'w');

    ------------------------------------------------------------------------
  • Mike at Mar 28, 2011 at 2:46 pm
    Edit report at http://pear.php.net/bugs/bug.php?id=18292&edit=1

    ID: 18292
    Updated by: mike@silverorange.com
    Reported By: hidenobu dot shinozuka at incs dot co dot jp
    Summary: S3 and Stream have a different option name.
    Status: Closed
    Type: Bug
    Package: Services_Amazon_S3
    Operating System: Irrelevant
    Package Version: 0.3.1
    PHP Version: Irrelevant
    Assigned To: gauthierm
    Roadmap Versions:
    New Comment:

    Release 0.3.2 contains a fix for this bug. Thanks for your report!


    Previous Comments:
    ------------------------------------------------------------------------

    [2011-03-28 15:36:00] gauthierm

    -Status: Assigned
    +Status: Closed
    This bug has been fixed in SVN.

    If this was a documentation problem, the fix will appear on pear.php.net
    by the end of next Sunday (CET).

    If this was a problem with the pear.php.net website, the change should
    be live shortly.

    Otherwise, the fix will appear in the package's next release.

    Thank you for the report and for helping us make PEAR better.

    Fixed in http://svn.php.net/viewvc?view=revision&revision=309787

    ------------------------------------------------------------------------

    [2011-03-28 15:07:52] gauthierm

    -Assigned To:
    +Assigned To: gauthierm


    ------------------------------------------------------------------------

    [2011-03-28 15:06:58] gauthierm

    -Status: Bogus
    +Status: Assigned
    -Operating System: Centos 5.5
    +Operating System: Irrelevant
    -PHP Version: 5.3.0
    +PHP Version: Irrelevant
    I looked over the code and the original reporter is correct. The
    variable name used in
    Stream.php is wrong. It's a bug, not a preference.

    As for the config name, I think we should continue to support
    'http_option' for
    backwards-compatibility, but also add 'http_config' as the word 'config'
    is used
    throughout HTTP_Request2.

    ------------------------------------------------------------------------

    [2011-03-27 16:07:05] till

    -Status: Open
    +Status: Bogus
    Sorry, but your problem does not imply a bug in PEAR itself. For a
    list of more appropriate places to ask for help using PEAR, please
    visit http://pear.php.net/support/ as this bug system is not the
    appropriate forum for asking support questions.

    Thank you for your interest in PEAR.

    One is an option, one is a class variable. Class variables follow
    camelcase, as do class methods.

    All other config options use an underscore, I don't think this is
    inconsistent - it would be if it was otherwise. If we change
    one option, we'll have to change all. Add to this, this package is
    actually used in many places and despite the lower
    version number I'm against breaking BC.

    ------------------------------------------------------------------------

    [2011-02-22 09:20:39] hiderer

    Description:
    ------------
    Http_option is defined "$httpConfig" at S3.php line 162, But in
    Stream.php specifying "httpOptions" or "http_options" at line 940
    to 944.

    "httpConfig" unified hope.

    Test script:
    ---------------
    Services_Amazon_S3_Stream::register('s3',
    array(
    'access_key_id' => $access_key,
    'secret_access_key' => $secret_key,
    'http_config' => array('proxy_host' => '172.24.xx.xx', 'proxy_port' =>
    8080)
    ));

    $streamS3 = fopen('s3://mybucket/mykey', 'w');

    ------------------------------------------------------------------------

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
grouppear-bugs @
categoriesphp
postedMar 27, '11 at 3:04p
activeMar 28, '11 at 2:46p
posts5
users2
websitepear.php.net

2 users in discussion

Mike: 4 posts Till: 1 post

People

Translate

site design / logo © 2022 Grokbase