Apache OpenOffice (AOO) Bugzilla – Full Text Issue Listing |
Summary: | Filename property in Base file connected to MS-Access database changed when file is closed | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | Base | Reporter: | ud <ulger> | ||||||
Component: | code | Assignee: | marc.neumann | ||||||
Status: | CLOSED FIXED | QA Contact: | issues@dba <issues> | ||||||
Severity: | Trivial | ||||||||
Priority: | P3 | CC: | drewjensen.inbox, issues, mdxonefour | ||||||
Version: | OOO320m8 | Keywords: | regression | ||||||
Target Milestone: | OOo 3.2 | ||||||||
Hardware: | PC | ||||||||
OS: | Windows XP | ||||||||
Issue Type: | DEFECT | Latest Confirmation in: | --- | ||||||
Developer Difficulty: | --- | ||||||||
Issue Depends on: | |||||||||
Issue Blocks: | 99999 | ||||||||
Attachments: |
|
Description
ud
2009-12-31 13:26:21 UTC
Created attachment 66925 [details]
attached screenshots
OOO320_m8 (RC1) Windows XP SP 3 and creating ODB files connected to MDB files, also checked with ACCDB files - see note. Confirmed '\' is being added to the file name. So "C:\TestData\TireRating.mdb" becomes "C:\TestData\TireRating.mdb\". (didn't check yet if it is on save or open..but will). Altered summary to match. NOTE it is worse with ACCDB file connections, however this particular problem also happens. Will open a separate issue for ACCDB because as opposed to here, where you can fix the error per session, there is no way to fix the problem there after the first time the file is closed (or opened again). The extraneous '\' is being added when the file is saved. The companion issue for accdb data type is http://www.openoffice.org/issues/show_bug.cgi?id=108040 Fixing this is not as easy as it might seem ... In general, for file-based database types, OOo adds the / at the end of the xlink:href value which points to the file or folder in question. The "file or folder" is exactly the problem here: It would be easily possible to fix the *import* code to strip this final slash. This would solve the problem here. However, speaking strictly, it's already wrong to write the "....accdb/" part. The URL here points to a file, so there must not be this final slash at all. However, the slash was introduced for fixing issue 96134, where it indeed is required, since the URL there denotes a folder. So, for a proper fix, we would need to distinguish between a reference to a folder and a reference to a file, which means extended the meta data for database types, which perhaps isn't really trivial ... At least that's nothing I would like to tamper with for 3.2 anymore, the more since even if this were fixed, issue 108040 would still prevent us from using the MSA2007 databases properly :-\ Okay, more investigations with Ocke's help showed that we can fix this properly, and pretty risk-less, without tampering with the configuration data. Will carry this to release@ooo. Adjusting target as announced on releases mailing list. fixed in CWS dba32k find more information about this CWS, like when it is available in the master builds, in EIS, the Environment Information System: http://eis.services.openoffice.org/EIS2/cws.ShowCWS?Path=DEV300%2Fdba32k fs->msc: please verify in CWS dba32k Created attachment 67021 [details]
patch as committed (for the record, for review, for ...)
verified in CWS dba32k find more information about this CWS, like when it is available in the master builds, in EIS, the Environment Information System: http://eis.services.openoffice.org/EIS2/cws.ShowCWS?Path=DEV300%2Fdba32k Patch verified. checked w/ xp, ooo320_R2 closing |