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

Fix deprecation warnings #614

Merged
merged 2 commits into from
Jul 3, 2024
Merged

Fix deprecation warnings #614

merged 2 commits into from
Jul 3, 2024

Conversation

patdel0
Copy link
Contributor

@patdel0 patdel0 commented Jul 2, 2024

This is to fix the deprecation warnings:

[DEPRECATION NOTICE] Docker Image Format v1 and Docker Image manifest
version 2, schema 1 support is disabled by def

and

image

How to Test

  1. Perform a search on the site
  2. Confirm you are seeing the deprecation errors
  3. Checkout this branch
  4. Run:
cd wp-content/themes/dxw-security-2017; composer update -W
  1. Refresh the page and confirm the errors are no longer present

@snim2
Copy link
Contributor

snim2 commented Jul 2, 2024

@patdel0 two quick questions, how are you seeing those PHP errors and why do we need to run composer?

@patdel0
Copy link
Contributor Author

patdel0 commented Jul 2, 2024

@snim2 Thanks for having a look at this one.

  • I am seeing them locally through query monitor. You can also see them on staging. This is happening whenever you search for a term on the site.

  • You might want to run composer to install the latest version of dxw/pagination.
    Although the composer files were updated,vendor/ is ignored in the repo. I reckon that when you switch to this branch, you'll find it's still running with v1.0 because of that.
    An alternative is to run ./script/setup again, but a composer run seems to be the faster option.

@snim2
Copy link
Contributor

snim2 commented Jul 2, 2024

@snim2 Thanks for having a look at this one.

  • I am seeing them locally through query monitor. You can also see them on staging. This is happening whenever you search for a term on the site.

OK, I'll check that tomorrow

  • You might want to run composer to install the latest version of dxw/pagination.
    Although the composer files were updated,vendor/ is ignored in the repo. I reckon that when you switch to this branch, you'll find it's still running with v1.0 because of that.
    An alternative is to run ./script/setup again, but a composer run seems to be the faster option.

The new version of the dependency should be packaged in the vendor.phar file, so that when the code is deployed it deploys with exactly the dependencies that the developer intended. composer install is usually for local development, where we have some special reason for needing the vendor/ directory (for example, because it might be easier to run a particular binary from the command line).

@snim2
Copy link
Contributor

snim2 commented Jul 3, 2024

This is what I see in Query Monitor btw

Screenshot 2024-07-02 at 21 40 55

@patdel0
Copy link
Contributor Author

patdel0 commented Jul 3, 2024

When searching for "private" does it have sufficient results to require pagination?

@snim2
Copy link
Contributor

snim2 commented Jul 3, 2024

When searching for "private" does it have sufficient results to require pagination?

Well, that was exciting. private only had 3 responses for me, but vuln had 3 pages.

patdel0 added 2 commits July 3, 2024 13:18
This is to fix the deprecation warning:
[DEPRECATION NOTICE] Docker Image Format v1 and Docker Image manifest
version 2, schema 1 support is disabled by def
@patdel0 patdel0 force-pushed the fix/deprecation-warnings branch from 7a66f68 to 55d49a4 Compare July 3, 2024 12:19
@patdel0 patdel0 merged commit e0c0a6f into develop Jul 3, 2024
9 checks passed
@patdel0 patdel0 deleted the fix/deprecation-warnings branch July 3, 2024 12:19
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.

2 participants