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

2491: Prevent URN map generation during indexing. #2499

Conversation

vitaliyboykocontributor
Copy link
Contributor

Added a check to avoid URN map generation when the project is in indexing (dumb) mode. A warning notification is shown to inform users that they need to wait until indexing is complete. This prevents potential errors or performance issues.

Description (*)

Fixed Issues (if relevant)

  1. Fixes Block "Generate URN Mapping" Button During Indexing #2491

Questions or comments

Contribution checklist (*)

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with integration/functional tests (if applicable)
  • All automated tests passed successfully (all builds are green)

Added a check to avoid URN map generation when the project is in indexing (dumb) mode. A warning notification is shown to inform users that they need to wait until indexing is complete. This prevents potential errors or performance issues.
Updated functionality to skip URN map generation while the project is indexing. This helps avoid potential errors and enhances performance stability. A warning notifies users to wait until indexing finishes.
…ssage-that-indexing-in-process

# Conflicts:
#	CHANGELOG.md
Refactored notification text formatting for clarity and consistency. This ensures users are properly informed to wait until indexing is complete before generating URN maps. Enhances user guidance and prevents unwanted actions during indexing.
Moved ExternalResourceManager initialization into the runnable to prevent execution during indexing. This ensures better stability and avoids potential premature resource access.
Moved PsiManager and MagentoComponentManager initialization into the runnable to delay their usage until the appropriate execution context. Added a suppression for PMD warning to ensure clarity and maintain compatibility with standards.
@VitaliyBoyko VitaliyBoyko merged commit 4250faa into magento:5.4.0-develop Feb 23, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

Block "Generate URN Mapping" Button During Indexing
3 participants