Go Premium for a chance to win a PS4. Enter to Win

x
  • Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 356
  • Last Modified:

ASPUpload - updating existing record using form param, PT II

this question is related to this (answered yesterday):
http://www.experts-exchange.com/Web/Web_Languages/ASP/Q_21918804.html

i thought i tested it but apparently something fell through the cracks!

i'm not able to update an existing record - so if i have a record who's ContentID is 65 and i try to update the filename in in the db when i do my upload to the server, taint happening-  if there's no filename the filename doesn't get added to the existing record - here's my code:

<!-- AspUpload Code samples: odbc_upload.asp -->
<!-- Invoked by odbc.asp -->
<!-- Copyright (c) 2001 Persits Software, Inc. -->
<!-- http://www.persits.com -->

<HTML>
<BODY>


<%
      Set Upload = Server.CreateObject("Persits.Upload")

      ' Capture files
      Upload.Save "E:\inetpub\wwwroot\smbb\newsite\news\pdfs\"

      ' Obtain file object
      Set File = Upload.Files("THEFILE")

      For Each File in Upload.Files

            datapath="E:\inetpub\wwwroot\smbb\smbb.mdb"
            strConn="Provider=Microsoft.Jet.OLEDB.4.0;Data Source=" & datapath
            Set objConn = Server.CreateObject("ADODB.Connection")
            objConn.Open strConn  
                  
  set rs = Server.CreateObject("ADODB.RecordSet")
     
Dim rsContentID__MMColParam
rsContentID__MMColParam = "ContentID"

SQL = "SELECT * FROM Content WHERE ContentID = " + Replace(rsContentID__MMColParam, "'", "''") + ""
 rs.Open SQL,objConn,1,2

  if rs.eof then
     rs.AddNew
  end if

          rs("filename") = File.ExtractFileName
          rs.Update
          rs.Close
          set rs = nothing
          objConn.Close
          set objConn = nothing

          Response.Write "Your PDF has been successfully uploadedd.  <a href='index.asp'>Click here</a> to return to the admin."
      Next
%>

</BODY>
</HTML>

Thanks!!!


0
phillystyle123
Asked:
phillystyle123
  • 6
  • 5
1 Solution
 
alorentzCommented:
>>if there's no filename the filename doesn't get added to the existing record

What's this mean?  No filename?  Why would you have that?

Also, may want to check your database to see if records are being added...because will do that is EOF.
0
 
phillystyle123Author Commented:
just rolled out of bed so i'm a tad less articulate than usual

records are being added to the db - i checked. i'm updating existing db records by updating the 'filename' field. basically, my client wants to upload pdfs and add links to those pdfs on the site-  one per record.  the pdf filenames are being added to records that have other fields, title, article, date, etc.

0
 
alorentzCommented:
OK, so please explain the problem again...as it relates to this code.

Also, always be sure to response.write your variables and SQL...so you know what's actually going on.  Sometimes we assume the code is performing as expected, but the code is not what we expect at all.
0
What does it mean to be "Always On"?

Is your cloud always on? With an Always On cloud you won't have to worry about downtime for maintenance or software application code updates, ensuring that your bottom line isn't affected.

 
phillystyle123Author Commented:
ok -

i'm passing along the form variable "ContenID"

i need the code to do this:

update 'filename' in db where db.ContentID=formparampassed.ContentID
0
 
phillystyle123Author Commented:
ugh  still struggling with this - how would i use response.write to display the form variable i'm trying to pass?

something like this?

Response.Write (rsContentID__MMColParam)

0
 
phillystyle123Author Commented:
ust did this

Response.Write rsContentID__MMColParam

and this is what i got so i'm guessing the form variable isn't being passed or it is but my code thinks the variable is "ContentID":

"Your PDF has been successfully uploadedd. Click here to return to the admin.ContentID"
0
 
alorentzCommented:
Stop using the stupid DW variables all together....they are annoying.    

contentid=request("contentid")

if contentid ="" then
   response.write "ContentID is empty"
else
  response.write "Content ID: " & contentid
end if


SQL = "SELECT * FROM Content WHERE ContentID = " & contentid
 rs.Open SQL,objConn,1,2

if rs.eof then
     rs.AddNew
  end if

          rs("filename") = File.ExtractFileName

...etc...
0
 
phillystyle123Author Commented:
thanks alorentz

using your code i'm getting the following error:

Request object error 'ASP 0208 : 80004005'

Cannot use generic Request collection

/newsite/admin/smbbnews/odbc_upload.asp, line 28

Cannot use the generic Request collection after calling BinaryRead.

here's your code built into the existing code:

<!-- AspUpload Code samples: odbc_upload.asp -->
<!-- Invoked by odbc.asp -->
<!-- Copyright (c) 2001 Persits Software, Inc. -->
<!-- http://www.persits.com -->

<HTML>
<BODY>


<%
      Set Upload = Server.CreateObject("Persits.Upload")

      ' Capture files
      Upload.Save "E:\inetpub\wwwroot\smbb\newsite\news\pdfs\"

      ' Obtain file object
      Set File = Upload.Files("THEFILE")

      For Each File in Upload.Files

            datapath="E:\inetpub\wwwroot\smbb\smbb.mdb"
            strConn="Provider=Microsoft.Jet.OLEDB.4.0;Data Source=" & datapath
            Set objConn = Server.CreateObject("ADODB.Connection")
            objConn.Open strConn  
                  
  set rs = Server.CreateObject("ADODB.RecordSet")
     
contentid=request("contentid")

if contentid ="" then
   response.write "ContentID is empty"
else
  response.write "Content ID: " & contentid
end if


SQL = "SELECT * FROM Content WHERE ContentID = " & contentid
 rs.Open SQL,objConn,1,2

  if rs.eof then
     rs.AddNew
  end if

          rs("filename") = File.ExtractFileName
          rs.Update
          rs.Close
          set rs = nothing
          objConn.Close
          set objConn = nothing

          Response.Write "Your PDF has been successfully uploadedd.  <a href='index.asp'>Click here</a> to return to the admin."
               Response.Write ContentID
      Next
%>

</BODY>
</HTML>

0
 
alorentzCommented:
Now we are getting somewhere <grin>

Change:

 contentid=request("contentid")

 to

 contentid=Upload.Form("contentid")

When uploading images in binary, you have to use the Upload component as the request object.   otherwise you use the request object.
0
 
alorentzCommented:
You can also see all that on the code sample page, that yuo've been too:

http://www.aspupload.com/codesample.html 
0
 
phillystyle123Author Commented:
thanks alorentz - all seems to be working - much appreciated
0

Featured Post

Free Tool: Port Scanner

Check which ports are open to the outside world. Helps make sure that your firewall rules are working as intended.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

  • 6
  • 5
Tackle projects and never again get stuck behind a technical roadblock.
Join Now