Skip to content

Commit a275560

Browse files
authored
Merge branch 'master' into master
2 parents 49c618c + 208a46c commit a275560

File tree

3 files changed

+18
-28
lines changed

3 files changed

+18
-28
lines changed

.versions

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,9 @@ mongo-id@1.0.6
4141
npm-mongo@2.2.30
4242
observe-sequence@1.0.16
4343
ordered-dict@1.0.9
44-
ostrio:autoform-files@2.0.1
44+
ostrio:autoform-files@2.0.2
4545
ostrio:cookies@2.2.2
46-
ostrio:files@1.8.1
46+
ostrio:files@1.8.2
4747
promise@0.8.9
4848
random@1.0.10
4949
reactive-dict@1.1.9

README.md

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -21,34 +21,35 @@ SimpleSchema.setDefaultMessages({
2121
```
2222
- Create your Files Collection (See [`ostrio:files`](https://github.com/VeliovGroup/Meteor-Files))
2323
```javascript
24-
var Images = new FilesCollection({
24+
const Images = new FilesCollection({
2525
collectionName: 'Images',
2626
allowClientCode: true, // Required to let you remove uploaded file
27-
onBeforeUpload: function (file) {
27+
onBeforeUpload(file) {
2828
// Allow upload files under 10MB, and only in png/jpg/jpeg formats
2929
if (file.size <= 10485760 && /png|jpg|jpeg/i.test(file.ext)) {
3030
return true;
3131
} else {
3232
return 'Please upload image, with size equal or less than 10MB';
3333
}
3434
}
35-
})
36-
37-
// if you want to make use of dburles:mongo-collection-instances
38-
// you need to create a 'parent' reference to the underlying collection
39-
Images.collection.filesCollection = Images;
35+
});
4036

4137
if (Meteor.isClient) {
4238
Meteor.subscribe('files.images.all');
4339
}
4440

4541
if (Meteor.isServer) {
46-
Meteor.publish('files.images.all', function () {
42+
Meteor.publish('files.images.all', () => {
4743
return Images.collection.find({});
4844
});
4945
}
5046
```
51-
__Note:__ If you don't use Mongo Collection instances, then the `Images` variable must be attached to *Global* scope. And has same name (*case-sensitive*) as `collectionName` option passed into `FilesCollection#insert({collectionName: 'Images'})` method, `Images` in our case.
47+
__Note:__ If you don't use Mongo Collection instances (`dburles:mongo-collection-instances`), then the `Images` variable must be attached to *Global* scope. And has same name (*case-sensitive*) as `collectionName` option passed into `FilesCollection#insert({collectionName: 'Images'})` method, `Images` in our case.
48+
49+
To start using `dburles:mongo-collection-instances` simply install it:
50+
```shell
51+
meteor add dburles:mongo-collection-instances
52+
```
5253

5354

5455
- Define your schema and set the `autoform` property like in the example below

lib/client/fileUpload.js

Lines changed: 6 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -11,10 +11,10 @@ Template.afFileUpload.onCreated(function () {
1111
atts: {}
1212
};
1313
}
14-
console.log(this.data);
14+
1515
this.collection = Mongo.Collection.get
1616
? Mongo.Collection.get(this.data.atts.collection).filesCollection
17-
: Meteor.connection._mongo_livedata_collections[this.data.atts.collection];
17+
: (global[this.data.atts.collection] || Meteor.connection._mongo_livedata_collections[this.data.atts.collection].filesCollection);
1818
this.uploadTemplate = this.data.atts.uploadTemplate || null;
1919
this.previewTemplate = this.data.atts.previewTemplate || null;
2020

@@ -47,15 +47,10 @@ Template.afFileUpload.helpers({
4747
return Template.instance().fileId.get() || this.value;
4848
},
4949
uploadedFile() {
50-
var collectionName = Template.instance().collectionName();
51-
var collection = Mongo.Collection.get
52-
? Mongo.Collection.get(collectionName).filesCollection
53-
: global[collectionName];
54-
if (!collectionName || !collection) return null;
55-
56-
var _id = Template.instance().fileId.get() || this.value;
50+
const template = Template.instance();
51+
var _id = template.fileId.get() || this.value;
5752
if (typeof _id !== 'string' || _id.length === 0) return null;
58-
return collection.findOne({_id:_id});
53+
return template.collection.findOne({_id:_id});
5954
}
6055
});
6156

@@ -77,13 +72,7 @@ Template.afFileUpload.events({
7772
},
7873
'change [data-files-collection-upload]'(e, template) {
7974
if (e.currentTarget.files && e.currentTarget.files[0]) {
80-
81-
var collectionName = template.collectionName();
82-
var collection = Mongo.Collection.get
83-
? Mongo.Collection.get(collectionName)
84-
: global[template.collectionName()];
85-
86-
var upload = collection.filesCollection.insert({
75+
const upload = template.collection.insert({
8776
file: e.currentTarget.files[0],
8877
streams: 'dynamic',
8978
chunkSize: 'dynamic'

0 commit comments

Comments
 (0)