Invoice Code issues

[WindowGuy]

Can not figure out the issue with this.
it saves the pdf but does not fill the records.

I followed a turtorial step by step and all worked until the last bit of code at the end....
Any help greatly appreciated.
FYI i am a beginner so may not understand references to items.

Sub SaveInvAsExcel()

Dim invno As Long
Dim custname As String
Dim amt As Currency
Dim dt_issue As Date
Dim term As Byte
Dim path As String
Dim fname As String

invno = Range("C3")
custname = Range("B10")
amt = Range("I41")
dt_issue = Range("C5")
term = Range("C6")
path = "C:\Users\GeorgeDoucette\Desktop\THERMOTEK\Invoices\"
fname = invno & " - " & custname
Sheet27.Copy

Dim shp As Shape

For Each shp In ActiveSheet.Shapes
    shp.Delete
Next shp

With ActiveWorkbook
    .Sheets(1).Name = "Invoice"
    .SaveAs Filename:=path & fname, FileFormat:=51
    .Close
End With

Set nextrec = Sheet28.Range("A1048576").End(xlUp).Offset(1, 0)

nextrec = invno
nextrec.Offset(0, 1) = custname
nextrec.Offset(0, 2) = amt
nextrec.Offset(0, 3) = dt_issue
nextrec.Offset(0, 4) = dt_issue + term

Sheet28.Hyperlinks.Add anchor:=nextrec.Offset(0, 7), Address:=path & fname & ".xlsx"


End Sub

 

[NoSparks]

Does it make any difference if you use Option Explicit to force declaration of all variables?

 

[WindowGuy]

Does it make any difference if you use Option Explicit to force declaration of all variables?

Like I said I’m a beginner no clue what that meant…

 

[NoSparks]

Ok, in my opinion, Option Explicit should always be used because it forces declaration of all variables.
What you have would work if you had declared nextrec as a Range.
Do a quick google search of Option Explicit, it can be set to be included at the top of all code modules.

In the VBA environment, Tools > Options > Editor
check box for Require Variable Declaration

 

[WindowGuy]

Ok, in my opinion, Option Explicit should always be used because it forces declaration of all variables.
What you have would work if you had declared nextrec as a Range.
Do a quick google search of Option Explicit, it can be set to be included at the top of all code modules.

In the VBA environment, Tools > Options > Editor
check box for Require Variable Declaration

I dim it as range and boom it worked like a charm. Thank you very much. I couldn’t figure the option explicit thing out but dim as range worked.

 

[NoSparks]

You're welcome.
I edited my last post (probably after you had already read it) and indicate where to set VBA to automatically add Option Explicit.