mongodb - Uploading multiuple images with Node.js using Multer S3/Amazon S3 -


overview: have figured out how allow users upload multiple images amazon s3 using node, express & multers3. code shows solution below. working well. issue on edit route. want users able edit or delete images. far way have been able have new images override previous filepaths.

this big problem because means user has reupload every single image every time try edit post. not good. think solution might create images array instead of having individual image paths. appreciate advice.

i've seen code creating array local storage i'm not sure how alter work amazon s3: 

if(req.files.length) {  for(var = 0; < req.files.length; i++) {  updatedlisting.moreimages.push() // needs push amazon s3 here somehow.  }  updatedlisting.save();

first show "listings" mongoose model: 

var listingsschema = new mongoose.schema({    name: string,    description: string,    image: string,    image2: string,    image3: string,    image4: string,    image5: string,    author: {       id:{        type: mongoose.schema.types.objectid,        ref: "user"       },        username: string,        email: string    } });

my code upload images amazon s3: 

aws.config.update({     secretaccesskey: 'xxxxxxxxxxxxxxxxxxxxxxxx',     accesskeyid: 'xxxxxxxxxxxxx',     region: 'us-east-2' });  var s3 = new aws.s3();  var storage =  multers3({             s3: s3,             bucket: 'mybucket',             key: function (req, file, cb) {                 var fileextension = file.originalname.split(".")[1];                 var path = "uploads/" + req.user._id + date.now() + "." + fileextension;                 cb(null, path);              },         })

here code find images , create new listing. you'll see below how hard coding check each filepath. believe better solution here loop unsure how update code.

var upload = multer({storage: storage}).any("image", 5);  router.post("/", middleware.isloggedin, function(req, res, next){          upload(req,res,function(err) {         if(err) {         console.log(err);         res.redirect('/')         }         var filepath = undefined;        var filepath2 = undefined;        var filepath3 = undefined;        var filepath4 = undefined;        var filepath5 = undefined;      if(req.files[0]) {         filepath = req.files[0].key;     }         if(req.files[1]) {         filepath2 = req.files[1].key;     }        if(req.files[2]) {         filepath3 = req.files[2].key;     }         if(req.files[3]) {         filepath4 = req.files[3].key;     }         if(req.files[4]) {         filepath5 = req.files[4].key;     }      var name = req.body.name;     var desc = req.body.description;     var image = filepath;     var image2 = filepath2;     var image3 = filepath3;     var image4 = filepath4;     var image5 = filepath5;       var newlistings = {name: name, description: description, image: image, image2: image2, image3: image3, image4: image4, image5: image5}      // create new listing , save db     listings.create(newlistings, function(err, newlycreated){         if(err){             console.log(err);         } else {  res.redirect("/listings");         }     });     });     });     });

and lastly part of update put page. real issue is. want user able upload new images without overriding old images. you'll see below filepaths same on post route overrides images regardless of how many images selected.

router.put("/:id", middleware.checklistingsownership, function(req, res){               upload(req,res,function(err) {             if(err) {             console.log(err);             res.redirect('/')             }             var filepath = undefined;            var filepath2 = undefined;            var filepath3 = undefined;            var filepath4 = undefined;            var filepath5 = undefined;          if(req.files[0]) {             filepath = req.files[0].key;         }             if(req.files[1]) {             filepath2 = req.files[1].key;         }            if(req.files[2]) {             filepath3 = req.files[2].key;         }             if(req.files[3]) {             filepath4 = req.files[3].key;         }             if(req.files[4]) {             filepath5 = req.files[4].key;         }          var name = req.body.name;         var desc = req.body.description;         var image = filepath;         var image2 = filepath2;         var image3 = filepath3;         var image4 = filepath4;         var image5 = filepath5;


Comments

Post a Comment

Popular posts from this blog

android - InAppBilling registering BroadcastReceiver in AndroidManifest -

i'm implementing inapp billing in android app. works well, however, i'm trying decouple broadcast receiver activity manifest. suggestion in android's trivialdrive sample: // important: dynamically register broadcast messages updated purchases. // register receiver here instead of <receiver> in manifest // because call getpurchases() @ startup, therefore can ignore // broadcasts sent while app isn't running. // note: registering listener in activity bad idea, done here // because sample. regardless, receiver must registered after // iabhelper setup, before first call getpurchases(). currently there's class extends broadcastreceiver : public class iabbroadcastreceiver extends broadcastreceiver { /** * intent action receiver should filter for. */ public static final string action = "com.android.vending.billing.purchases_updated"; private final iabbroadcastlistener mlistener; public iabbroadcastreceiver(iabbroadcastlistener listener) { m...
Read more

python Tkinter Capturing keyboard events save as one single string -

is possible store keyboard events 1 string? code below stores 1 char , prints it. card reader or bar code reader, contains collection of character/ string not 1 character @ time. goal save char pressed text variable. from tkinter import * root = tk() def key(event): text= event.char text+= event.char print ("pressed", text) def callback(event): frame.focus_set() print ("clicked at", event.x, event.y) frame = frame(root, width=100, height=100) frame.bind("<key>", key) frame.bind("<button-1>", callback) frame.pack() root.mainloop() currently, creating text variable , everytime key() function gets called, , text stores last character typed. you can define text module level variable , , use module level text inside key function - text = '' def key(event): global text text+= event.char print("pressed", text)
Read more

sql server - Why does Linq-to-SQL add unnecessary COUNT()? -

here tables: create table customer ( cid int primary key, name varchar(32) ) create table ord ( oid int primary key, cid int foreign key references customer, address varchar(20) ) here's linq-to-sql statement: var aaa = c in db.customer select new { c, o = c.ord.tolist() }; and here's inexplicable query generated linq-to-sql: select [t0].[cid] [cid], [t0].[name] [name], [t1].[oid] [oid], [t1].[cid] [cid2], [t1].[address] [address], (select count(*) [dbo].[ord] [t2] [t2].[cid] = [t0].[cid]) [value] [dbo].[customer] [t0] left outer join [dbo].[ord] [t1] on [t1].[cid] = [t0].[cid] order [t0].[cid], [t1].[oid] i want understand how rid of count(*) part. it's entirely uncalled for! linq needs count(*) in order determine number of entries returned o = c.ord.tolist() .
Read more