Problems with auto numbering....

kjicha

New Member
Joined
Feb 29, 2016
Messages
41
I am having an issue when the numbering with my macro, when I get to 1.9 after it is 1.9 and it goes to the next row it just puts 1.1 It needs to be 1.10 Same when it gets to 1.19 it just goes to 1.2 instead of 1.20.

Any ideas?

Below is my code:

Sub WBSNumbering()
'From http://j.modjeska.us/?p=31
'Renumber tasks on a project plan
'Associate this code with a button or other control on your spreadsheet
'Layout Assumptions:
'Row 1 contains column headings
'Column A contains WBS numbers
'Column B contains Task description, with appropriate indentation
'Some text (here we assume "END OF PROJECT") delimits the end of the task list
On Error Resume Next
'Hide page breaks and disable screen updating (speeds up processing)
Application.ScreenUpdating = False
ActiveSheet.DisplayPageBreaks = False
'Format WBS column as text (so zeros are not truncated)
ActiveSheet.Range("A:A").NumberFormat = "#"
Dim r As Long 'Row counter
Dim depth As Long 'How many "decimal" places for each task
Dim wbsarray() As Long 'Master array holds counters for each WBS level
Dim basenum As Long 'Whole number sequencing variable
Dim wbs As String 'The WBS string for each task
Dim aloop As Long 'General purpose For/Next loop counter
r = 3 'Starting row
basenum = 0 'Initialize whole numbers
ReDim wbsarray(0 To 0) As Long 'Initialize WBS ennumeration array
'Loop through cells with project tasks and generate WBS
Do While Cells(r, 3) <> ""
'Ignore empty tasks in column B
If Cells(r, 3) <> "" Then
'Skip hidden rows
If Rows(r).EntireRow.Hidden = False Then
'Get indentation level of task in col B
depth = Cells(r, 3).IndentLevel
'Case if no depth (whole number master task)
If depth = 0 Then
'increment WBS base number
basenum = basenum + 1
wbs = CStr(basenum)
ReDim wbsarray(0 To 0)
'Case if task has WBS depth (is a subtask, sub-subtask, etc.)
Else
'Resize the WBS array according to current depth
ReDim Preserve wbsarray(0 To depth) As Long
'Repurpose depth to refer to array size; arrays start at 0
depth = depth - 1
'Case if this is the first subtask
If wbsarray(depth) <> 0 Then
wbsarray(depth) = wbsarray(depth) + 1
'Case if we are incrementing a subtask
Else
wbsarray(depth) = 1
End If
'Only ennumerate WBS as deep as the indentation calls for;
'so we clear previous stored values for deeper levels
If wbsarray(depth + 1) <> 0 Then
For aloop = depth + 1 To UBound(wbsarray)
wbsarray(aloop) = 0
Next aloop
End If
'Assign contents of array to WBS string
wbs = CStr(basenum)
For aloop = 0 To depth
wbs = wbs & "." & CStr(wbsarray(aloop))
Next aloop
End If
'Populate target cell with WBS number
Cells(r, 2).Value = wbs
'Get rid of annoying "number stored as text" error
Cells(r, 2).Errors(xlNumberAsText).Ignore = True
'Apply text format: next row is deeper than current
If Cells(r + 1, 3).IndentLevel > Cells(r, 3).IndentLevel Then
Cells(r, 2).Font.Bold = False
Cells(r, 3).Font.Bold = False
'Else (next row is same/shallower than current) no format
Else
Cells(r, 2).Font.Bold = False
Cells(r, 3).Font.Bold = False
End If

'Special formatting for master (whole number) tasks)
If Cells(r, 3).IndentLevel = 0 Then
Cells(r, 2).Font.Bold = True
Cells(r, 3).Font.Bold = True
End If

'Formatting color of row with no indents
If Cells(r, 3).IndentLevel = 0 Then
Cells(r, 1).Interior.ColorIndex = 48
Cells(r, 2).Interior.ColorIndex = 48
Cells(r, 3).Interior.ColorIndex = 48
Else
If Cells(r, 3).IndentLevel = 1 Then
Cells(r, 1).Interior.ColorIndex = 15
Cells(r, 2).Interior.ColorIndex = 15
Cells(r, 3).Interior.ColorIndex = 15
Else

If Cells(r, 3).IndentLevel > 1 Then
Cells(r, 1).Interior.ColorIndex = 0
Cells(r, 2).Interior.ColorIndex = 0
Cells(r, 3).Interior.ColorIndex = 0
End If
End If
End If


End If
End If

'Go to the next row
r = r + 1
Loop
End Sub


Additional coding:

Private Sub Worksheet_Change(ByVal Target As Range)
On Error GoTo quit
Application.EnableEvents = False
With Target
'If .Count = 1 Then
If .Column = 3 Then
.WrapText = True
'If .Offset(0, -2).Value = "" Then .Offset(0, -2).Value = Range("A1") + 1
End If
'End If
End With
quit:
Application.EnableEvents = True
End Sub
 
Last edited:
I changed it back to what it was and then I had to format the column as Text and that seemed to have fixed the issue. Thank you.
 
Upvote 0

Excel Facts

Highlight Duplicates
Home, Conditional Formatting, Highlight Cells, Duplicate records, OK to add pink formatting to any duplicates in selected range.

Forum statistics

Threads
1,225,155
Messages
6,183,214
Members
453,151
Latest member
Lizamaison

We've detected that you are using an adblocker.

We have a great community of people providing Excel help here, but the hosting costs are enormous. You can help keep this site running by allowing ads on MrExcel.com.
Allow Ads at MrExcel

Which adblocker are you using?

Disable AdBlock

Follow these easy steps to disable AdBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the icon in the browser’s toolbar.
2)Click on the "Pause on this site" option.
Go back

Disable AdBlock Plus

Follow these easy steps to disable AdBlock Plus

1)Click on the icon in the browser’s toolbar.
2)Click on the toggle to disable it for "mrexcel.com".
Go back

Disable uBlock Origin

Follow these easy steps to disable uBlock Origin

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back

Disable uBlock

Follow these easy steps to disable uBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back
Back
Top