Merge pull request #53 from launchdarkly/eb/ch93198/dynamodb-prefix

make prefix concatenation in DynamoDB consistent with other SDKs

Author: eli-darkly

src/LaunchDarkly/Impl/Integrations/DynamoDbFeatureRequester.php

@@ -26,7 +26,7 @@ public function __construct($baseUri, $sdkKey, $options)
         $this->_client = new DynamoDbClient($dynamoDbOptions);
 
         $prefix = isset($options['dynamodb_prefix']) ? $options['dynamodb_prefix'] : '';
-        $this->_prefix = ($prefix != null && $prefix != '') ? ($prefix . '/') : '';
+        $this->_prefix = ($prefix != null && $prefix != '') ? ($prefix . ':') : '';
     }
 
     protected function readItemString($namespace, $key)

tests/DynamoDbFeatureRequesterTest.php

@@ -53,7 +53,7 @@ protected function putItem($namespace, $key, $version, $json)
         self::$dynamoDbClient->putItem(array(
             'TableName' => self::TABLE_NAME,
             'Item' => array(
-                'namespace' => array('S' => self::PREFIX . '/' . $namespace),
+                'namespace' => array('S' => self::PREFIX . ':' . $namespace),
                 'key' => array('S' => $key),
                 'version' => array('N' => strval($version)),
                 'item' => array('S' =>  $json)