Code design flaw

[samueldcs]

RLAPI-v3-OOP/src/Uploader.php

Line 11 in b873773

if(isset($_GET['bucket']) && !is_null($_GET['bucket'])) //Switch from array_key_exists to isset

You are not supposed to access the request body directly from the class, instead, pass the value from the API endpoint (index.php).

Example (index.php):

if(isset($_GET['bucket'] && !is_null[$_GET['bucket'])) {
    $uploader = new RLME\Uploader($_GET['bucket']);
}

And if the bucket is not set (isset() == false), set a default value on the constructor.

Example (Uploader.php)

function __construct($bucket = 'owoapi') {}

If you want, I can create a PR solving this issue

[gtsatsis]

We're currently not working on the uploader, and that's just bootstrap code.

If you want to make a PR, you could, but if not, it'll be fixed when we start work on the uploader

[samueldcs]

Just keep that in mind and fix it when you can.