Skip to content

Commit

Permalink
Updated dependencies like ostrio:files etc.
Browse files Browse the repository at this point in the history
Thanks to developers of dependencies and xet7 !

Related veliovgroup/Meteor-Files#847
  • Loading branch information
xet7 committed Aug 15, 2022
1 parent c091fa5 commit 7c61985
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 7 deletions.
12 changes: 6 additions & 6 deletions .meteor/versions
@@ -1,4 +1,4 @@
accounts-base@2.2.3
accounts-base@2.2.4
accounts-oauth@1.4.1
accounts-password@2.3.1
aldeed:collection2@2.10.0
Expand All @@ -10,11 +10,11 @@ allow-deny@1.1.1
arillo:flow-router-helpers@0.5.2
audit-argument-checks@1.0.7
autoupdate@1.8.0
babel-compiler@7.9.0
babel-compiler@7.9.2
babel-runtime@1.5.1
base64@1.0.12
binary-heap@1.0.11
blaze@2.6.0
blaze@2.6.1
blaze-tools@1.1.3
boilerplate-generator@1.7.1
caching-compiler@1.2.2
Expand Down Expand Up @@ -94,8 +94,8 @@ meteorhacks:subs-manager@1.6.4
meteortesting:browser-tests@1.3.5
meteortesting:mocha@2.0.3
meteortesting:mocha-core@8.0.1
minifier-css@1.6.0
minifier-js@2.7.4
minifier-css@1.6.1
minifier-js@2.7.5
minifiers@1.1.8-faster-rebuild.0
minimongo@1.8.0
modern-browsers@0.1.8
Expand Down Expand Up @@ -123,7 +123,7 @@ ongoworks:speakingurl@1.1.0
ordered-dict@1.1.0
ostrio:cookies@2.7.2
ostrio:cstorage@4.0.1
ostrio:files@2.0.1
ostrio:files@2.2.1
ostrio:i18n@3.2.0
pascoual:pdfkit@1.0.7
peerlibrary:assert@0.3.0
Expand Down
6 changes: 5 additions & 1 deletion models/attachments.js
Expand Up @@ -40,13 +40,17 @@ Attachments = new FilesCollection({
debug: false, // Change to `true` for debugging
collectionName: 'attachments',
allowClientCode: true,
/* Commenting out because this custom namingFunction did not work:
https://github.com/veliovgroup/Meteor-Files/issues/847
namingFunction(opts) {
const filenameWithoutExtension = opts.name.replace(/(.+)\..+/, "$1");
const filenameWithoutExtension = opts.meta.name.replace(/(.+)\..+/, "$1");
const ret = opts.meta.fileId + "-original-" + filenameWithoutExtension;
// remove fileId from meta, it was only stored there to have this information here in the namingFunction function
delete opts.meta.fileId;
return ret;
},
*/
storagePath() {
const ret = fileStoreStrategyFactory.storagePath;
return ret;
Expand Down

1 comment on commit 7c61985

@dr-dimitru
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @xet7
I've seen veliovgroup/Meteor-Files#847
And I'd like to jump in order to help resolving this one.
Could you explain initial purpose of custom namingFunction() hook? Since you have commented it out — it works without it in the expected way?

Please sign in to comment.