Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix absolutize URL for several cases #861

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Fix absolutize URL for several cases #861

wants to merge 4 commits into from
+122 −23

Commits on Apr 5, 2024

  1. Fix absolutize URL for several cases 

    There were a number of bugs related to the fact that `Item::get_links()` and `Item::get_base()` call each-other, making a nice mess during initialisation. See tests.

Furthermore, the standard Atom `self` link was not supported, wrongly falling back to `alternate`. In the same PR because otherwise the tests from both PRs would fail.
    @Alkarex
    Alkarex committed Apr 5, 2024
    Configuration menu
    Copy the full SHA
    76a4f62 View commit details
    Browse the repository at this point in the history

  2. Minor style

    @Alkarex
    Alkarex committed Apr 5, 2024
    Configuration menu
    Copy the full SHA
    ceae5eb View commit details
    Browse the repository at this point in the history

  3. Fix PHPStan

    @Alkarex
    Alkarex committed Apr 5, 2024
    Configuration menu
    Copy the full SHA
    8541e5d View commit details
    Browse the repository at this point in the history

Commits on Apr 10, 2024

  1. Improved comment

    @Alkarex
    Alkarex committed Apr 10, 2024
    Configuration menu
    Copy the full SHA
    13a398d View commit details
    Browse the repository at this point in the history