[Confluence] Bug - Page ID is ignored when Space is provided

[ddnimara]

Bug Summary

Current logic in confluence/connector.py makes it so page_id is not considered when space is provided, effectively indexing the entire space.

The code in question is here.

The logic in place

        elif space:
            # if no cql_query is provided, we will use the space to fetch the pages
            cql_page_query += f" and space='{quote(space)}'"
        elif page_id:
            if index_recursively:
                cql_page_query += f" and ancestor='{page_id}'"
            else:
                # if neither a space nor a cql_query is provided, we will use the page_id to fetch the page
                cql_page_query += f" and id='{page_id}'"

Potential fix

Considering this point, we should swap the elifs so that space is ignored if a specific page_id is provided.

This would also ensure that the current code state reflects what is present in the documentation which states:

Page ID (optional): Specific page ID to index - leave empty to index the entire space (e.g., 131368 for a specific page)

implying that in order to index the entire space, you should leave the page_id field empty (which conversely implies that if you do not leave it empty, it will not parse the entire space).