Apply suggestions from code review

GromNaN · alcaeus · jmikola · web-flow · commit 687d94c2baa5 · 2024-04-10T10:25:55.000+02:00
Co-authored-by: Andreas Braun &lt;git@alcaeus.org&gt;
Co-authored-by: Jeremy Mikola &lt;jmikola@gmail.com&gt;
diff --git a/docs/examples/lambda-aws/index.php b/docs/examples/lambda-aws/index.php
@@ -6,15 +6,10 @@
 
 $uri = getenv('MONGODB_URI') ?: throw new RuntimeException('The MONGODB_URI environment variable is not set');
 $client = new Client($uri);
-$planets = $client
-    ->selectDatabase('sample_guides')
-    ->selectCollection('planets')
+$planets = $client->sample_guides->planets
     ->find(
         [],
-        [
-            'sort' => ['orderFromSun' => 1],
-            'typeMap' => ['root' => 'array'],
-        ],
+        ['sort' => ['orderFromSun' => 1]],
     );
 
 ?>
@@ -26,7 +21,7 @@
 <body>
     <ul>
         <?php foreach ($planets as $planet) : ?>
-            <li><?= $planet['name'] ?></li>
+            <li><?= $planet->name ?></li>
         <?php endforeach ?>
     </ul>
 </body>
diff --git a/docs/examples/lambda-aws/serverless.yml b/docs/examples/lambda-aws/serverless.yml
@@ -21,8 +21,3 @@ functions:
         layers:
             - ${bref-extra:mongodb-php-83}
 
-# Exclude files from deployment
-package:
-    patterns:
-        - '!node_modules/**'
-        - '!tests/**'
diff --git a/docs/tutorial/lambda-aws.txt b/docs/tutorial/lambda-aws.txt
@@ -1,5 +1,5 @@
 ============================================
-Deploy to Serverless AWS Lambda with BrefPHP
+Deploy to AWS Lambda with Bref
 ============================================
 
 .. default-domain:: mongodb
@@ -19,21 +19,21 @@ Overview
 In this tutorial, you will deploy a simple PHP application with the MongoDB PHP extension,
 and connect to an Atlas cluster using AWS IAM authentication.
 
-Pre-requisites
+Prerequisites
 --------------
 
-Before you begin, you must install bref on your machine. You can follow the
+Before you begin, you must install Bref on your machine. You can follow the
 `official documentation to setup bref <https://bref.sh/docs/setup>`_.
 
-Install the mongodb extension
+Install the MongoDB extension
 -----------------------------
 
 By default, the bref layer is compiled with PHP and a few extensions. Additional extensions
 are provided in additional layers.
 
 Start by creating a new directory for your project and install the required mongodb
 and bref dependencies. This project will be a bare minimum PHP web application that
-connects to a MongoDB cluster, no framework is used, but you can adapt the code to
+connects to a MongoDB cluster. No framework is used, but you can adapt the code to
 your needs.
 
 .. code-block:: bash
@@ -43,7 +43,7 @@ your needs.
     vendor/bin/bref init
 
 
-A very simple web page have been created. To validate the deployment, you can start
+The file ``index.php`` has been created. To validate the deployment, you can start
 by deploying this default application.
 
 .. code-block:: bash
@@ -89,9 +89,9 @@ Atlas supports passwordless authentication with AWS credentials. In any Lambda f
 AWS sets environment variables that contains the access token and secret token with
 the role assigned to deployed function.
 
-Set Up `unified AWS Access <https://www.mongodb.com/docs/atlas/security/set-up-unified-aws-access/>`
+Set up `unified AWS Access <https://www.mongodb.com/docs/atlas/security/set-up-unified-aws-access/>`
 
-1. Open the lambda function in the AWS console
+1. Open the Lambda function in the AWS console
 2. In "Configuration > Permission", copy the "Role name"
 3. Open the MongoDB Atlas project
 4. Go to "Security > Database Access"
@@ -109,7 +109,7 @@ Update the ``serverless.yml`` file to pass the environment variable ``MONGODB_UR
 
     provider:
         environment:
-            MONGODB_URI: "mongodb+srv://cluster0.abcdefg.mongodb.net/"
+            MONGODB_URI: "mongodb+srv://cluster0.example.mongodb.net/"
 
 
 The value can be found in "Atlas > Deployment > Database > Connect". Select "3. AWS IAM".
