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.

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

Urls test #1

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Urls test #1

wants to merge 5 commits into from

Conversation

MarikaBBB
Copy link
Owner

@MarikaBBB MarikaBBB commented Jan 16, 2025

Description:

This PR is in relation to Issue dxw#19 and implements unparse_url() function to complement PHP's native parse_url(). While parse_url() parses a URL into its components, PHP does not provide a built-in way to reconstruct the URL, which is why this function is needed.

Change:

  • I created a PHPUnit test in urls_test.php to check the functionality before writing the actual implementation into the src folder.
  • I created Urls.php file in the src/ folder to implement the unparse_url() function.

Reference:

Consideration:

  • This implementation is based on the Stackoverflow solution (see link above)

How to test:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant