Describe the bug

When using the Notion::database() method to retrieve pages in a database, a HandlingException is thrown if it encounters a Rollup that refers to a Select, and that Select is empty.

The exception is thrown inside FiveamCode\LaravelNotionApi\Entities\Properties\Select::fillFromRaw(). Exception message:

"The property-type is select, however the raw data-structure does not reprecent this type. Please check the raw response-data."

To Reproduce

Laravel 8.60, OS MacOS 11.6 (Bug Sur), PHP 8.0.14.

1. Create 2 databases in Notion, DB1 and DB2.
2. In DB1, create a Select property titled "Select". Enter a couple of options.
3. In DB2, create a Relation to DB1. Call it "Relation".
4. In DB2, create a Rollup that uses the relation to DB1, and its Select property. Call it "Rollup".
5. Create a row in DB1 but leave the Select empty. Title it "A".
6. Create a row in DB2, Title it "X", and select "A" in its Relation property.

Now use the following code to read the DB2 database:

$notion = new Notion(secret_key);
$results = $notion->database(id_for_DB2)->query();

Expected behavior

All rows from DB2 should be returned in $results. Instead, HandlingException is thrown with the message

"The property-type is select, however the raw data-structure does not reprecent this type. Please check the raw response-data."

Hey, I have some Notion records (pages) where they have properties but also have this text inside the page:

Reading the Notion API documentation I figured out this is a rich text https://developers.notion.com/reference/rich-text however I am unable to find how to use it..

Do you know if this is supported by the Notion API or not? Because I saw in your code you already have created a RichText property but couldn't find how to call that function.. Thanks!

Hey hey,

hope this is not a newbie issue. First of all thanks for the amazing job. I'm having some issues with pagination.

I'm using the package within laravel. When querying a database it seems that the returned results are a Laravel Collection and the method getNextCursor doesn't exist. here's the error I'm getting:

 BadMethodCallException 

  Method Illuminate\Support\Collection::getNextCursor does not exist.

  at vendor/laravel/framework/src/Illuminate/Macroable/Traits/Macroable.php:113
    109▕      */
    110▕     public function __call($method, $parameters)
    111▕     {
    112▕         if (! static::hasMacro($method)) {
  ➜ 113▕             throw new BadMethodCallException(sprintf(
    114▕                 'Method %s::%s does not exist.', static::class, $method
    115▕             ));
    116▕         }
    117▕ 

and my sample code:

            if(!$offset)
            {
                $tasks=$notion->database($databaseId)
                ->sortBy($sortings)
                ->limit(5)
                ->query()
                ->asCollection();
            } else {
                $tasks=$notion->database($databaseId)
                ->sortBy($sortings)
                ->offset($offset)
                ->limit(5)
                ->query()
                ->asCollection();
            }

            $offset= $tasks->getNextCursor();

Any help would be hugely appreciated :)

Hi. Thanks for this great package!

Describe the bug I'm trying to create a page in a database, and following the example here: https://5amco.de/docs/0.5.0/examples/update_create/create_page

Basically this didn't work because the method createInDatabase doesn't exist in the Notion class: Notion::createInDatabase($databaseId, $page);

To Reproduce

Try to run Notion::createInDatabase($databaseId, $page);

Laravel: 8.56

Expected behavior

A page is successfully created in Notion.

Exceptions

None.

Additional context

I did notice that the Pages class does have the createInDatabase, I tried the following code, but that returned a "Bad Request" error.

use FiveamCode\LaravelNotionApi\Notion;
use FiveamCode\LaravelNotionApi\Entities\Page;

$notion = new Notion($token);
$page = new Page();
$page->setTitle("Title", "Date Prop");
$notion->pages()->createInDatabase($dbId, $page);

Describe the bug When adding a Youtube Video to a newly created Notion page the block is correctly added with its caption, the origin link sends to the correct video in Youtube, but the youtube player is not rendered correctly:

To Reproduce $newBlock = Video::create("https://www.youtube.com/watch?v=KLPZzf-wwlE",$video['title']); $notion->block($notionPage->getId())->append($newBlock);

Bonus Quesion Is there a way to create the page using a template? :)

Thank you so much for your hard work on this great library

fix: check if content is set for rollup properties

• specific rollup properties can be null, which lead to issues of setting the content of the property
• this fix checks the existence of content and skips setting the content, if it is null (e.g. empty selects, multi_selects etc.)
• fixes issue: Rollup that refers to an empty Select property throws exception during database query

I'm trying to retrieve a simple list of pages in a database. In Notion, I have manually dragged and re-ordered these to be in a specific order.

When I retrieve them simply as a collection...

Notion::database($databaseId)->query()->asCollection();

... and loop through, they are not in the same order as in Notion.

Yes I know I can specify explicit ordering, but I don't have any field/property that I can order on. My goal is to let the user manually drag to order items, and display them in that same order in my app.

Is this possible?

Is your feature request related to a problem? Please describe. At the moment when a Date property is added to a Page, it's automatically set with date and time, even if time is actually not needed.

Describe the solution you'd like It would be awesome if we could choose wether to display time or not! Because displaying time in Notion when not needed is a bit "polluting".

Describe alternatives you've considered In the meantime I make sure I set my Datetime with 0 in hours, minutes and seconds.

Additional context What I would like: Instead of:

Describe the bug

The union type float|int in the function definition of Filter::numberFilter() is flagged as a syntax error, when using PHP 7.4.

Union types are only valid in PHP8.0 and above, but PHP 7.4 is being supported in composer.json.

To Reproduce Call Filter::numberFilter() while using PHP 7.4.

Expected behavior There should not be a syntax error.

Exceptions This works fine in PHP8.0. So maybe remove all use of union types or make PHP8.0 the minimum requirement?

Additional context None.

Without a cast, it will be sent as an object and Notion will return a 400 error:

Bad Request: (validation_error) (body failed validation: body.start_cursor should be a string or `undefined`, instead was `{}`.)  

• implemented static creation-function for all modifiable blocks
• implemented append-function to Block-Endpoint
• implemented tests-function, in which all block-creation-functions are tested
• added todo/warning for notion-version-change to test-function

• add assertions for exception codes
• change the FiveamCode\LaravelNotionApi\Exceptions\NotionException::instance method to use $payload['responseData']['status'] field if it exists in exception code. 0, otherwise.

Describe the bug

When I use Filter::rawFilter to query a Notion database on a status property type, a validation error is returned.

To Reproduce

Set the filter and send the query:

$filters = new Collection(); $filters->add(Filter::rawFilter("Status", ["status" => [Operators::EQUALS => "Ready"]] ));

$notion = new Notion($access_token)

$rows = $notion->database($id) ->filterBy($filters) ->sortBy($sortings) ->query() ->asCollection();

Expected behavior

A filtered results collection is returned.

Exceptions

Trace can be found here: https://flareapp.io/share/pPv1vqYm#F58

Additional context

Describe the bug When a NotionException is thrown, the code is always 0.

To Reproduce No reproduction needed. See code: https://github.com/5am-code/laravel-notion-api/blob/4f75db65b22cb600ffdfba14eab49b1ef36bee54/src/Exceptions/NotionException.php#L47-L51

Expected behavior The exception should contain the HTTP Status code.

Exceptions . Additional context .

See my PR https://github.com/5am-code/laravel-notion-api/pull/88

Search-Endpoint while searching using a cursor is broken.

Here it is correct: https://github.com/5am-code/laravel-notion-api/blob/4f75db65b22cb600ffdfba14eab49b1ef36bee54/src/Endpoints/Database.php#L70

Bug: generates an empty array/object: https://github.com/5am-code/laravel-notion-api/blob/20a7fcc7cfce1a479a4218705a179d1d3f48d296/src/Endpoints/Search.php#L64

Error message

Bad Request: (validation_error) (body failed validation: body.start_cursor should be a string or `undefined`, instead was `{}`.)

I'd like to use the compound filters (currently a TODO comment) when querying a database. Specifically, I would like to use an "and" filter to add two filters at the same time.

Really just opening the issue to give me a place to track it whenever it gets implemented - love this package thanks for all your excellent work on it.