Summary: | FILEOPEN: HTML with .xls extension is opened in Writer/Web when opening from StartCenter | ||
---|---|---|---|
Product: | LibreOffice | Reporter: | Dave Miller <justdave> |
Component: | Libreoffice | Assignee: | Maxim Monastirsky <momonasmon> |
Status: | RESOLVED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | medium | CC: | acasalsi, irenec39, momonasmon |
Version: | 4.1.2.1 rc | Keywords: | regression |
Hardware: | Other | ||
OS: | All | ||
See Also: |
https://bugs.freedesktop.org/show_bug.cgi?id=68903 https://bugs.freedesktop.org/show_bug.cgi?id=88701 |
||
Whiteboard: | target:4.3.0 | ||
i915 platform: | i915 features: | ||
Attachments: | test file |
Description
Dave Miller
2014-01-16 05:23:38 UTC
Already confirmed with 4.1.2.1 at Bug 68903 comment 2. (In reply to comment #0) > In this case, selecting one of the Excel filetypes from the selector results > in an error trying to open the file Of course, since it's not an Excel file, just renamed HTML. Dave, please attach a test file to this bug if possible. (In reply to comment #2) > Dave, please attach a test file to this bug if possible. Should be reproducible with *any* HTML renamed to .xls Dave, so true, but having a test file unifies the testing case and makes sure we receive identical results. Otherwise this bug can become very confusing very quickly. I think we can agree it's a good idea to avoid that, no? Created attachment 92211 [details]
test file
For the record, I included a test case in the initial description. You just had to copy/paste it into a file and put an .xls extension on the filename. :) In fact, that appears to be exactly what Maxim did to create the file he attached. :) So, this is unfortunately by design. HTML file format is a generic format not specific to a single application type. If you want an HTML file to be opened in Calc, you can open it from Calc's file open menu. Having said that, since we do something similar for CSV, we could try to handle this as well. The right way to handle it is to have the HTML detection service inspect the file extension, and decide in which application to open the file in, the same way we do for CSV file format. The caveat is that the HTML detection code is split between writer and calc currently. The necessary first step would be to combine these split html detection codes into a central place, then put in a fix there. In fact, the csv detection code used to be split in the writer and calc detection codes as well. I had moved that into where it is now. We should do the same for html. Maxim Monastirsky committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=cc2893834d8ac699dbb38b152f21f17f3debb06b related: fdo#73682 Introduce HTML detection service The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. This is now fixed on master. Nice work Maxim! *** Bug 76234 has been marked as a duplicate of this bug. *** *** Bug 76114 has been marked as a duplicate of this bug. *** Maxim, the problem is within 4.3.2 build as well (Windows)? I thought this was fixed for 4.3? I change the extension to .html and "open with" CALC and it opens fine. There are no patches out there for 4.3.2. Hi msaum, As a general rule NEVER reopen old bugs. Instead: - Open a new bug. - Attach a test document, that shows the bug. (The bug is mostly fixed, but there could be a problem with a specific file. Therefore I can't say anything without seeing the file you're using.) - Add me to the CC list of that bug. Thanks. |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.