zotero / zotero-word-for-mac-integration

Zotero Word for Mac integration
https://www.zotero.org/support/word_processor_plugin_usage
Other
24 stars 12 forks source link

Removing field codes doesn't work #4

Closed simonster closed 12 years ago

simonster commented 12 years ago

at least in one of my manuscripts

mronkko commented 12 years ago

Confirmed. There is a hard-coded path into the compiled code

t3080-la0003% git clone https://github.com/zotero/zotero-word-for-mac-integration.git Cloning into zotero-word-for-mac-integration... remote: Counting objects: 1520, done. remote: Compressing objects: 100% (710/710), done. remote: Total 1520 (delta 912), reused 1378 (delta 770) Receiving objects: 100% (1520/1520), 2.06 MiB | 319 KiB/s, done. Resolving deltas: 100% (912/912), done. t3080-la0003% grep -R "/Users/simon" * Binary file zotero-word-for-mac-integration/resource/libzoteroMacWordIntegration.dylib matches t3080-la0003%

The error message that word gives is

[Exception ... '"The operation couldn't be completed. (OSStatus error -1708.) @[removeCode:IUsersl simon I Desktop I Development I FS I zote ro -word-formac- integrationlbuildlsrclfield.m:287)' when calling method: [zoterolntegrationFiel d:: removeCode)" nsresult: ·ox80S7001e (NS_ERRORXPCjS TH REW_STRING)" location: "JS frame :: chrome:llzoterolcontentlxpcoml integrat ion.js :: ::line 1020• data: no)

simonster commented 12 years ago

I tried to fix this in the latest version, and it works for me in the manuscript where it previously failed. What version of Word do you have? Does it work in a new document with just one field?

The path is there because I use the __FILE__ preprocessor instruction so that I know where the exceptions come from. I could try to chop off the path part and hope the compiler optimizes it out, but it's not related to the issue here.

mronkko commented 12 years ago

Word 2011. The problem appears in both new documents and a document that I have had for a while. (Maybe created with Zotero 2.0, but I cannot remomber exactly)

mronkko commented 12 years ago

To clarify: Creating a new document with just one field and then choosing "remove field codes" results in error.

simonster commented 12 years ago

Are you sure that your clone is up to date? (If you were previously on the scriptingBridge branch, you will have to switch to master.) I don't think the present code should be able to give an error message with that line number.

mronkko commented 12 years ago

I believe that I am currently not using the git version, but the released version. In Zotero standalone the version of the word integration is 3.5.

simonster commented 12 years ago

The current version available on the website is 3.5.2. If you install the plugin manually into Zotero Standalone (as opposed to using the bundled plugin), it doesn't automatically update.