mattcasters opened a new pull request, #7382:
URL: https://github.com/apache/hop/pull/7382

   # Walkthrough — Column Selection Before Excel Export
   
   For issue #7118
   
   ## Changes Made
   
   ### 
[TableViewExportToExcelToolbarButton.java](file:///home/matt/git/mattcasters/hop/plugins/transforms/excel/src/main/java/org/apache/hop/ui/core/widgets/TableViewExportToExcelToolbarButton.java)
   
   - Added `PKG` class constant and `BaseMessages` / `EnterSelectionDialog` 
imports.
   - Before building the workbook, the method now:
     1. Collects all column names from `tableView.getColumns()`.
     2. Opens an `EnterSelectionDialog` in **multi-select mode** with all 
columns pre-selected.
     3. If the user cancels or selects nothing, the export aborts gracefully.
     4. Otherwise, only the user-selected columns are written to the Excel 
header and data rows.
   - The `#` row-number column is always included automatically (not offered 
for deselection).
   - The auto-size loop now sizes only `selectedColumns.size() + 1` columns 
instead of all.
   
   ### 
[messages_en_US.properties](file:///home/matt/git/mattcasters/hop/plugins/transforms/excel/src/main/resources/org/apache/hop/ui/core/widgets/messages/messages_en_US.properties)
   
   Added two new i18n keys:
   ```properties
   ExcelWidget.ExportColumnSelection.Title=Select columns to export
   ExcelWidget.ExportColumnSelection.Message=Select the columns to include in 
the Excel export:
   ```
   
   ## Validation
   
   | Check | Result |
   |-------|--------|
   | `./mvnw clean install -pl plugins/transforms/excel -am -DskipTests` | ✅ 
Passed |
   | `./mvnw spotless:apply -pl plugins/transforms/excel` | ✅ Passed |
   | Manual verification: field selection dialog in Hop GUI | ✅ Verified |
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to