VBA DoUntil Issues

[chicagoland8]

Hi all--

having an issue with some VBA code. The main goal is to print the desired sheets based on the numbers entered. Ex. user enters 123, 456, and 789. This will open sheets of those names and print them out. I am trying to design it so that if they enter a number that doesn't have a corresponding workbook named after it, it will skip this one and go on to the next (and doesn't enter "Printed" in the cell next to it.). With my current code, if there is one incorrect number, it works fine. However, if there are two in a row, it stops there and doesn't continue even if there are legitimate numbers after.

Please let me know if that makes sense, here is my code:

Please Login or Register  to view this content.

[patel45]

attach a sample file for testing

[chicagoland8]

Here's my file--may be hard to test since it's opening files connected to a network drive here

Thanks so much!

[Norie]

Try this.

Please Login or Register  to view this content.

Oh, and when posting code please use code tags.

[chicagoland8]

Thanks a lot, that works perfectly for the print part!

When trying to use it with the openboms macro, it gets an out of range error...the only thing I changed was get rid of

Please Login or Register  to view this content.

and change the "Printed" to "Opened." Would I need to change something else for this to work? Thanks so much again for your help!

[Norie]

The OpenBOMs sub is never referred to in the code you posted and I can't see how changing anything in the posted code that would affect the sub.

Why did you get rid of those lines of code?

They are meant to call the sub Prints, as you were in the original code, and then close the workbook that's just been opened, as you do in the original code.

Changing 'Printed' to 'Opened' shouldn't affect anything at all, it would only change the value shown next to Part #.

[chicagoland8]

Sorry about the confusion--the OpenBoms was in my attached excel sheet but I didn't mention it in my post. I was able to figure it out though; This part was meant to just open the documents, not prints as in the previous code. I had to reactivate the previous workbook in order for it to work properly.

Thanks again, all solved!

[Norie]

Glad you've got it working, though I admit I'm not sure what exactly you are doing.

I thought you were openinh each workbook the user had specified, printing from it and then closing it.

Now it sounds like you are opening all the workbooks at the same time.

Anyway, it's working that's the important thing.

[chicagoland8]

You are absolutely correct--I was opening each workbook, printing, and closing; that part worked perfectly!

However, as in my posted example, I also had a second button / VBA code that just opens (doesn't print) each workbook specified. That's the one I slightly modified the first code you posted and wasn't working until I re-activated the "Daily Prints" workbook.

Thanks again!