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: disable metadata export for APIP #2507

Closed

Conversation

bartlomiejmarszal
Copy link
Contributor

When exporting APIP package, disable metadata export.

Metadata export for APIP will be developed soon so it will be enabled.

How to test:

  1. Export Package with APIP Content Package
  2. Unzip and open file. Make sure metadata has not been included.
  3. Import APIP package

Expected behavior
Item APIP package does not contain metadata.
Importing APIP Package is successful.

@bartlomiejmarszal bartlomiejmarszal changed the base branch from master to develop May 30, 2024 12:05

//APIP does not support metadata at this moment
if(!isset($options['apip']) && $options['apip'] !== true){
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn it be OR instead of AND ? If apip is not set OR apip has value false ? Then do metadata stuff?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both conditions are required. First if apip is not set you should go into if block. And if it is set it should be other than true to enter block. If I use OR it can lead engine to check both conditions and in second condition if it is not set it will cause fatal error.

Copy link

github-actions bot commented May 31, 2024

Front-end summary Node 18

💯 Total ✅ Passed ⏭️ Skipped ❌ Failed
286 286 0 0

Copy link

Version

Target Version 30.11.1
Last version 30.11.0

There are 0 BREAKING CHANGE, 0 feature, 3 fixes

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.

4 participants