Skip to content

chore: remove unused imports from view_unpushed_bundles.jsp#35362

Merged
nollymar merged 1 commit intomainfrom
fix/remove-unused-imports-view-unpushed-bundles
Apr 17, 2026
Merged

chore: remove unused imports from view_unpushed_bundles.jsp#35362
nollymar merged 1 commit intomainfrom
fix/remove-unused-imports-view-unpushed-bundles

Conversation

@nollymar
Copy link
Copy Markdown
Member

@nollymar nollymar commented Apr 16, 2026

Fixes #35373

Removed stale imports that are no longer referenced in the file.

Proposed Changes

  • Removed 9 unused <%@page import="..."%> declarations from view_unpushed_bundles.jsp

Checklist

  • Tests — N/A (JSP cleanup, no logic change)
  • Translations — N/A
  • Security Implications Contemplated — N/A (import removal only)

Removed stale imports that are no longer referenced in the file.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added the Area : Backend PR changes Java/Maven backend code label Apr 16, 2026
@claude
Copy link
Copy Markdown
Contributor

claude bot commented Apr 16, 2026

Claude finished @nollymar's task in 31s —— View job


Claude finished @nollymar's task in 38s —— View job


Clean. All 9 removed imports (CaseFormat, Language, EnvironmentAPI, Environment, Contentlet, PublishAuditUtil, DotContentletStateException, Logger, DotPublisherException) are confirmed unused in the file body. No functional change, no risk.

@nollymar
Copy link
Copy Markdown
Member Author

@claude can you create an issue from this PR?

@claude
Copy link
Copy Markdown
Contributor

claude bot commented Apr 17, 2026

Claude finished @nollymar's task in 3m 20s —— View job


Creating GitHub Issue

  • Analyze PR content
  • Attempt to create GitHub issue

Unfortunately, I'm unable to create GitHub issues in this CI environment — the gh issue create command is blocked by the runner sandbox and requires interactive approval that isn't available here.

Issue Details (ready to create manually)

You can create the issue by running this command locally:

gh issue create \
  --repo dotCMS/core \
  --title "chore: remove unused imports from view_unpushed_bundles.jsp" \
  --body "## Summary

Remove 9 stale/unused imports from \`view_unpushed_bundles.jsp\` that are no longer referenced in the file.

## File

\`dotCMS/src/main/webapp/html/portlet/ext/contentlet/publishing/view_unpushed_bundles.jsp\`

## Unused Imports to Remove

- \`com.google.common.base.CaseFormat\`
- \`com.dotmarketing.portlets.languagesmanager.model.Language\`
- \`com.dotcms.publisher.environment.ajax.EnvironmentAPI\`
- \`com.dotcms.publisher.environment.bean.Environment\`
- \`com.dotmarketing.portlets.contentlet.model.Contentlet\`
- \`com.dotcms.publisher.util.PublishAuditUtil\`
- \`com.dotmarketing.exception.DotContentletStateException\`
- \`com.dotmarketing.util.Logger\`
- \`com.dotcms.publisher.pusher.DotPublisherException\`

## Notes

- Pure code cleanup, no functional changes
- None of these imports are referenced in the file body
- Related: PR #35362"

Alternatively, you can create the issue directly on GitHub using the details above, then add a Fixes #<issue-number> line to the PR description to satisfy the issue-linking check.

@nollymar nollymar added this pull request to the merge queue Apr 17, 2026
Merged via the queue into main with commit d373a56 Apr 17, 2026
50 of 51 checks passed
@nollymar nollymar deleted the fix/remove-unused-imports-view-unpushed-bundles branch April 17, 2026 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI: Safe To Rollback Area : Backend PR changes Java/Maven backend code

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

chore: remove unused imports from view_unpushed_bundles.jsp

2 participants