[18.0][ADD] dms_spreadsheet: create/edit OCA spreadsheets in DMS#465
Open
[18.0][ADD] dms_spreadsheet: create/edit OCA spreadsheets in DMS#465
Conversation
Contributor
|
tests need to succeed :) |
Bridges OCA DMS (dms.file) with spreadsheet_oca so users can create and edit spreadsheets directly from the DMS file manager. Features: - dms.file gains a `handler` field (selection: 'spreadsheet') - Inherits spreadsheet.abstract for collaborative editing via bus + revisions - Auto-sets handler/mimetype on create when content is application/o-spreadsheet - open_spreadsheet() returns action_spreadsheet_oca client action - get_spreadsheet_data() bridges dms.file content to spreadsheet raw JSON - Storage bridge: _compute/_inverse_content delegates to spreadsheet_binary_data for spreadsheet-type files; standard DMS content path for all others - CreateSpreadsheetWizard: create named spreadsheet in any directory, opens editor immediately on confirm - Directory form + kanban: "New Spreadsheet" button pre-fills directory_id - File form: "Open Spreadsheet" button shown when handler == 'spreadsheet' - File list/search: "Spreadsheets" filter - Server action on dms.file for "New Spreadsheet" in file manager action menu Depends: dms, spreadsheet_oca (OCA/spreadsheet 18.0) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
dnplkndll
force-pushed
the
18.0-add-dms_spreadsheet
branch
from
edef871 to
0358c80
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds
dms_spreadsheet, a bridge module that integrates OCA DMS withspreadsheet_ocaso users can create and edit spreadsheets directly from the DMS file manager.What it does
handler='spreadsheet'/ MIME typeapplication/o-spreadsheetaction_spreadsheet_ocaclient action (fromspreadsheet_oca)content↔spreadsheet_binary_databridgespreadsheet.abstractmixinDependencies
dms(this repo)spreadsheet_oca(OCA/spreadsheet)Tests
test_create_sets_handler— MIME type → handler auto-set on createtest_open_spreadsheet_action— correct client action tag + paramstest_get_spreadsheet_data_empty/_with_content/_readonly— data + modetest_content_bridge_compute/_inverse— content ↔ spreadsheet_binary_data synctest_mimetype_forced_for_spreadsheet_handler— mimetype locked toapplication/o-spreadsheettest_wizard_create_spreadsheet— wizard creates file + opens editortest_wizard_default_get_from_context— directory pre-fill from contexttest_non_spreadsheet_content_unaffected— regular files unchangedtest_get_spreadsheet_data_corrupted— malformed binary data handlingRelated
spreadsheet_oca+spreadsheet.abstract)