Printable View
Thanks for your assistance.
The problem is with this file:
catalog\includes\templates\template_default\common\tpl_tabular_display.php on line 36
line 36 is:
<?php echo '<' . $cell_type . $c_params . '>'; ?><?php echo $list_box_contents[$row][$col]['text'] ?><?php echo '</' . $cell_type . '>' . "\n"; ?>
Still not sure what the prob is.
Thanks.
What is the actual text from the error log in the cache folder?Quote:
Originally Posted by elninomago
[07-Apr-2011 06:38:18] PHP Fatal error: Maximum execution time of 30 seconds exceeded in C:\websites\xxxxxxx.com\catalog\includes\templates\template_default\common\tpl_t abular_display.php on line 36
Hmmm...ok...I'm not an expert at reading error log files, but based on the info you provided (both the error log and the line info from the file it points to), I'm going to guess that this file isn't loading:Quote:
Originally Posted by elninomago
admin/includes/boxes/extra_boxes/super_orders_customers_dhtml.php
I think we're going to need Diva's help on this one. But while we're waiting for her to chime in, here are a couple things I would try if your situation was my own:
1. Re-upload that file and see what happens. Maybe the original got corrupted during the upload on the install.
2. If that doesn't work, check to make sure all files were uploaded to their proper places, and then
3. Re-Read the ReadMe to make sure you followed all instructions exactly. If you did not, then uninstall (per the uninstall instructions) and reinstall.
These are GENERAL instructions as the usual cause for failed module installs is that shopowners forget to rename certain store folders in the module files to match their store.Quote:
Originally Posted by elninomago
Quote:
Originally Posted by elninomago
Let me start with I don't think you've provided enough or the right information to get a sense of what is going on with your install..Quote:
Originally Posted by elninomago
Referring to the text highlighted in red above: Is this the error message you said you see when accessing Super Orders or is this from a Zen Cart error log file?????
If this is coming from a Zen Cart error log file this is unrelated to Super Orders.
If you are getting some kind of error message when you go to Customers > Super Orders, you need to post that error message.. It's not enough to simply say "I get an error".
and this is also my guess that this is an incomplete SO3 install..Quote:
Originally Posted by ScriptJunkie
Hi All!
I'm receiving this error in my admin debug log:
[07-Apr-2011 13:39:38] PHP Warning: stripslashes() expects parameter 1 to be string, array given in /home/server/public_html/admin/super_orders.php on line 131
Also, I am not getting the shipping detail line to show anywhere. But order total shows the shipping costs added in.
Any ideas? Oh, I do have TY Tracker installed and turned on/linked.
Okay,
I understand what you were saying before about the template override system. I get it.
I checked all of the files that were part of the installation pack.
Here's what I found:
1. In the Install directory, no files are actually named the same as my zen cart files. ie. no files would have been overwritten.
2. In the require Core Edit files, the following files were overwritten:
Admin/Includes/general.js
Admin/Includes/Modules/orders_download
Includes/Classes/order.php
Just thought I'd give you more info.
Right, but because these files are outside of the override system, they won't require a custom template. However, this is why it's very important to keep a copy of the original core file as a back up - in case you need to uninstall this mod. Keeping a zipped copy of the zen cart version you're using should be sufficient for that purpose.Quote:
Originally Posted by elninomago
In the meantime, your real problem seems to be an incomplete installation - meaning that one or more files were not uploaded, were corrupted on upload, were uploaded to the wrong folder, or some other mistake in the installation process was made.
I would go back and re-read the ReadMe file to make sure you've understood everything correctly. Be sure and read ALL the tabs in the ReadMe, but especially these: Installation & Settings, Configure & Features, and Add-Ons Integration.
This is particularly important if you have any of these add ons:
- Edit Orders
- USPS Autofill Click-N-Ship
- FedEx Automated Shipping Labels
- Ty Package Tracker
I believe your best bet is to uninstall by deleting all the files you uploaded and then running the uninstall sql file included in the package, and then reinstall - but only after you completely read the Read Me file.
Also, note that there are different directions depending on whether this is a new install or an upgrade from a previous version.
Lastly, if uninstall/reinstall does not work, then in your next post, please include what version of SO3 you're using, what version of Zen Cart you're running, what version of PHP is installed on your server, what you've tried so far to make it work and the results of those tries.
Thanks, I will try that.
Hi...thanks for monitoring this mod...just upgraded to 1.3.9h and this is our first install of super orders.
I'm getting this error on my super orders detail page beneath the product listing.
I've carefully redone all my edits for all mods and checked my file uploading and everything looks good. Mods include the newest Edit Orders, Ty tracker (what a bugger that code is to integrate :( twice) click and ship, and info at a glance:
Code:1054 Unknown column 'total_weight' in 'field list'
in:
[select total_weight from orders where orders_id = '10727']
If you were entering information, press the BACK button in your browser and re-check the information you had entered to be sure you left no blank fields.
Would help if you told us EXACTLY what you were trying to do when you got the error..Quote:
Originally Posted by chadderuski
You'll need to explain what this is..Quote:
Originally Posted by chadderuski
This is an error from the USPS Click N Ship module.. You will need to post on their support forum to get assistance with this..
Quote:
Originally Posted by athena
Okay,
REmoved, reinstalled, still not working.
What I found while looking over the files section in the readme file is that I'm missing this file from the install pack:
super_orders_30/1_Install_Files/YOUR_ADMIN_FOLDER/super_customers.php
no super_customers.php file.
Is this still a requirement for v3?
Thanks
super_customers.php Not needed..Quote:
Originally Posted by elninomago
I need you to provide a LOT more detail on what you mean when you say "still not working".
What happens?? in DETAIL please... and by detail do not simply say "I get an error message".. Tell us WHAT the message says!!! Please..
Seems like we need a reminder about this..:smile:
From page 1 of this support thread: http://www.zen-cart.com/forum/showpo...19&postcount=3
Quote:
Originally Posted by DivaVocals
When I'm in my Admin, I go to Customers, then Super Orders and I get:
HTTP 500 Internal Server Error.
the Website cannot display the page.
Most likely causes:
•The website is under maintenance.
•The website has a programming error.
Thanks.
Have you installed any other admin mods?? (Edit Orders, Ty Package Tracker, USPS Click-N-Ship, etc)Quote:
Originally Posted by elninomago
What happens when you go to the default orders.php???
HOW did you install the Super Orders SQL??
I should add that the Configuration, Super Orders tabs work just fine. Maybe I don't have something configured proplerly within that section?
I'm running Zen Cart V1.3.9b
Thanks
**sigh**Quote:
Originally Posted by elninomago
I still need answers to the questions I previously posted..
Lastly 500 error may not be related to Super Orders at all.. Have you taken a look at this FAQ:Quote:
Originally Posted by DivaVocals
Why am I getting "500 Internal Server Error"?
There is no longer an Orders section under the Customer tab.
I just thought this got omitted when installing Super Orders. (Why a need for both?)
Thanks.
I followed all of the instructions carefully in the readme file.
including running the SQL file in the Install SQL Patches section.
and these questions??Quote:
Originally Posted by DivaVocals
The only other mods I have installed are the apsona mod, ups XML, and the ceon credit card mod.
Like I said previously, the orders selection under customers is not there since I've installed the super orders mod.
Okay.. it is MOST LIKELY that the issue you are having is because you have some or all of Super Orders installed incorrectly..Quote:
Originally Posted by elninomago
You need to check over ALL of your updates and modifications very carefully.. In fact I would start by reversing the "Optional Edits" which hides the default orders.php.
After you un-hide the default orders.php file, you need to post the results of what happens when you go to the default orders.php page..
Also.. the 500 error may not be related to Super Orders at all.. Have you taken a look at this Zen Cart FAQ: Why am I getting "500 Internal Server Error"?
How would I go about removing the optional edits? Do you mean the optional files from the optional files folder I. The install pack?
Then how do I injure orders.php?
Thanks for all of your assistance
Please read the "Optional Files Use & Installation" section of the readme.. You need to UNDO this part of the install to display the default orders.php file..Quote:
Originally Posted by elninomago
I've undone the Optional files folder ftp by deleting all the files from that particular part of the install.
Now I can't even get to my admin. HTTP 500 Error.
I upload them again and I can access the admin.
the customers , Super Orders section doesn't work still, but I can replace the words super_orders.php with orders.php and I can see the orders page again.
Did you read the readme file for this part of the install as I suggested?? You can't simply delete this file.. This would have indeed disabled your admin as you reported.. When I said UNDO this part of the install, I meant you need to undo the edits that are made to this file.Quote:
Originally Posted by elninomago
This is not an error caused by Super Orders at all.. You are getting this error because Super Orders is installed incorrectly. I can't see your store so I couldn't begin to GUESS what the issue is. I suggest two things..Quote:
Originally Posted by elninomago
One:
Roll back to a version of your store before you installed Super Orders (you DID back up the store before you installed RIGHT??) and start completely over with your SO3 install..
Two:
Read the FAQ I suggested SEVERAL times..
Also please don't take this the wrong way, but you may want to consider paying someone to help you with this install.. SO3 is a straightforward install, and if you are getting errors it is because this is installed incorrectly.. Someone looking at your store could get this straightened out in short order..Quote:
Also.. the 500 error may not be related to Super Orders at all.. Have you taken a look at this Zen Cart FAQ: Why am I getting "500 Internal Server Error"?
here's an edited screenshotAttachment 9084Quote:
Originally Posted by DivaVocals
This is what you should see, and it mimics the same behavior that is seen in the default orders.php page..Quote:
Originally Posted by highlander
To clarify this a little:Quote:
Originally Posted by DivaVocalsThis is what you should see, and it mimics the same behavior that is seen in the default orders.php page..Quote:
Originally Posted by highlander
The behavior of what's displayed in the tax included columns is controlled here:
Admin >Configuration > My Store > Display Prices with Tax in Admin
This is DEFAULT Zen Cart behavior. Since SO3 (and all previous versions) are based on the default orders.php, the SO3 equivalent page, super_orders.php retains many of the same characteristics as it's default orders.php cousin.
The columns you see displayed in super_orders.php are the same columns displayed in orders.php
New install of 1.3.9h
I followed the super orders recommendations and installed Edit orders first, Ty tracker, usps click and ship - all worked great:
I'm about to remove super orders but first some questions:
What is the advantage of super orders over edit orders? Can't see any through the admin. Why is it recommended to install both?
are there known issues with the click and ship mod and super orders? have uninstalled click and ship reluctantly as it did work, but super orders had issues with it.
so my plan is to un install super orders and re install click and ship
any insights or suggestions?
Just to clarify my situation:
Yes, I also have an install of the info at a glance mod...but did that separate from the usps auto fill mod.
OK, CHECK...Code:USPS.com Auto-Fill Integration
If you have USPS.com Auto-Fill v1.4 installed you can now enable USPS.com Auto-Fill fields in the Super Orders system.
1. Install USPS.com Auto-Fill v1.4
2. DO NOT USE the super_order.php files that come with USPS.com Auto-Fill v1.4. They are for an older version of Super Orders. Super Orders v3.0 already has the correctly integrated USPS.com Auto-Fill v1.4 code included.
NO, installed clean from the info at a glance mod....Code:If you are using the "Info At a Glance" flavor of the super_orders.php files that come with USPS.com Auto-Fill v1.4
I did not run any additional scripts. Just the super_orders_install_v30.sqlCode:we have provided updated versions of that super_orders.php file in the a2_USPS_W_INFO_AT_A_GLANCE folder. Super Orders v3.0 already has the correctly integrated USPS.com Auto-Fill v1.4 code included
3. Go to Configuration > Super Orders. Set the "USPS.com Auto-Fill Switch" to "True".
This will enable the USPS.com Auto-Fill v1.4 fields/buttons and make them available in Super Orders. DO NOT activate this flag if you do NOT have USPS.com Auto-Fill v1.4 installed or you WILL get errors in your store!!!
I only installed the basic super_orders files and also made the modifications to the core file requested.
any help understanding these instructions better appreciated.
No one is answering over in the click and ship thread....thanks for monitoring this mod.
I was a little premature...sorry - in mod install mode for too many weeks now....
spent some time testing a few things.
Want to keep Super Orders.
Would like to know if I ran the sql statement (s) appropriately as stated above in my install details.
My understanding of the instructions was that if everything was a clean install from the newest mod into the newest zencart that only the basic super orders sql need be run.
Right? Wrong? I un installed the Auto fill function and my super order errors went away, but
Super orders lets us record virtual refunds and real additional payments with the ability to enter cc info.
Although total due is correct, new info does not show up at default 1.3.9h orders.php where refunds are concerned...did not reflect in totals, just in notes. additional Payment does not show up in default orders.php Can't install the bypass on this yet as the user is scared of the new superorders...but that is on the list to do.
SO Does not save cc info :)
Saves expiration date on card :(
Super Orders print data sheet function only brings up a blank page.
I get the same thing on this page. There is a error but I just cleared my cache folder out.Quote:
Originally Posted by athena
Can you visit your /catalog/cache folder (or your /cache folder) and look for a .log file with the word admin in it? If you have multiple .logs, visit the control panel and try to visit the print data sheet again. Then, go to your FTP folder and visit the cache file and copy/paste the contents of the most recent file (should be the same time you tried to open the page) and paste it here.
Since you didn't post what the error is, I can only guess it's one that has been reported already in this support thread.. (which you would NOT see in the Zen Cart error logs)Quote:
Originally Posted by retched
A search of this thread for the following terms "super_data_sheet.php" will yield a posted fix for the super_data_sheet.php file.
Ahh that one. I did that one already.. I thought it was a different file, nevermind here.Quote:
Originally Posted by DivaVocals
DivaVocals. Thanks for your help and your suggestions.
We would not have included this code if there were known issues.. It was tested thoroughly before SO3 was released and there were no issues in our testing (there were three of us testing this code).Quote:
Originally Posted by athena
If the USPS Click-N-Ship module is installed correctly you only have to turn on the USPS Click-N-Ship flag in SO3 to use it with SO3. Common error in installing mods include skipped/missing files and not running the module's included SQL files (or running them incorrectly -- running SQL files using phpMyAdmin vs the Zen Cart SQL installer)
If there are issues with USPS Click-N-Ship it's most likely that a step in installing the USPS Click-N-Ship module was skipped/overlooked (which is why I directed you back to the proper support thread for help). Understandably we can't provide support for other modules in this support thread.
thank you for the reply...
what i'm not clear on is if i already installed usps click and ship and just ran the basic super_orders sql when i installed super orers?
is that the right way to do it?
All we did with S03 was provide an easy integration path for several popular modules.Quote:
Originally Posted by athena
Instead of having to merge the othe modules code into the super_ordes.php file, we've done that heavy lifting for the shop owner. All the shop owner has to do is to install the other module and "turn it on" in SO3. The admin flags in SO3 make the required fields/updates for the appropriate module "active" in the super_orders.php page.
However for these modules to work correctly with Super Orders they have to be installed correctly. Errors on the super_orders.php page can ONLY mean that either the module is NOT installed or not installed CORRECTLY.
From page 1 of this support thread.. Please note what's highlighted in red.
Quote:
- EASY integration* with Edit Orders, USPS Click-N-Ship, Fedex Automated Labels, and Ty Package Tracker
- More navigation options
- More shortcut icons on the orders listing screen. Now there is a corresponding shortcut icon on each order line that matches the buttons that appear in the sidebox
*Edit Orders, USPS Click-N-Ship, Fedex Automated Labels, and Ty Package Tracker MUST be installed to use these built-in integration features
Hi, I'm seeing this error in the cache log files:
[13-Apr-2011 11:44:37] PHP Warning: Missing argument 3 for update_status(), called in /home/XXXXXXX/public_html/parts/XXXX/includes/classes/super_order.php on line 230 and defined in /home/XXXXXXX/public_html/parts/XXXX/includes/functions/extra_functions/common_orders_functions.php on line 101
This happened when we marked some old orders as "Completed". Everything in the database looks good, and Super Orders seems to be functioning correctly, just throwing a few stray errors.
We're running V1.3.9h with Super Orders v. 3.0 and "Edit Orders" v3.03. I just thought I'd give a 'heads up'.
Thanks for the great add-on!
Mike
Loving the module, certainly making everything easier - but have a slight issue when it comes to printing a pdf. (or viewing) I'm getting a blank page when trying to print the PDF packing slips, and am kind of scratching my head here. (There are no php errors that I can find in the cache.) Most likely missing something really obvious, but?
I am having an issue with the Batch Form Printing. When I try to print multiple invoices I get an new window with all the selected invoices smashed onto one screen with only the store address showing for each invoice. When I try to print the invoices, I get one page with one the store address for each selected invoice.
Is there a way to fix this so that each selected invoice prints on its own page?
I had this as well.Quote:
Originally Posted by mdbcfc
In the admin folder under Includes > Functions > Extra Functions
Edit the common_orders_functions.php file:
Change line 77 from:
to:PHP Code:function update_status($oID, $new_status, $customer_notified, $comments = '', $track_id1, $track_id2, $track_id3, $track_id4, $track_id5) {
PHP Code:function update_status($oID, $new_status, $customer_notified = 0, $comments = '', $track_id1 = '', $track_id2 = '', $track_id3 = '', $track_id4 = '', $track_id5 = '') {
Hi All,
Just a quick question about the new version 3.0 regarding shipping labels, are you able to edit the position of the text because we use Australian Express post satchel and would like to use labels rather than wrighting on them.
Note that we use different size satchels so we would require 3 different layouts to be able to choose from. Would this be possible.:D
Hi,
I want to install this mod. I've downloaded it and found the installation help file (super_orders_readme.html), but the file only outlines the first section, Purpose & Aim. It doesn't have the rest of the sections that go through the installation process. Am I looking at the right file? Or is there another one?
I don't want to try to install this mod without thoroughly reading the install instructions first.
Thank you!
If you are reading the readme from the zip file you won't "see" the rest of the readme.. Unzip the package to access the full readme..Quote:
Originally Posted by WriteAtHome
Yes you could edit the shipping label.. adding 3 different layouts would require a custom change.Quote:
Originally Posted by Bruce1952
Batch invoices open the selected invoices in one window with each invoice in a separate frame. You need a browser which support printing each frame separately as a print option (like Firefox and/or Internet Explorer)Quote:
Originally Posted by cycochuck
Not sure.. double check your install make sure all files are in place..Quote:
Originally Posted by Ubique17
Hi! First of all, thanks for the great addon.
I'm trying to make a little customisation to my super_batch_orders.php - specifically the pdf packingslips masterlist function. I'm trying to make the list also display attribute titles for each particular item.
I've been having a bit of a poke around the code, and i assume i'm needing to modify the code within the while loop called at around line 409. What i want to know is what string or array should I be using to retrieve the attribute title?
Thanks - Hope you can help!
[edit]
i'd also love to be able to display the type of postage selected and total weight if this is possible!
Sorry I can't help you with this.. As I've posted here previously, the PDF packinglist code was generously donated to the Super Order module by a community member.. It is AS IS code, and I'm afraid I don't know enough about it to make more than a few cursory modifications (formatting changes mostly).
That said when my time permits I do have a new version of SO3 which includes some changes to the appearance of attributes on the PDF packinglist (also generously donated by the same member). Dunno if it will serve your needs..
I've been a tad busy lately, and haven't had time to work on volunteer projects.. However, I am hoping to have some time Friday or Saturday to finish up and submit SO3. (Have Edit Orders mods that need to go up together..)
Quote:
Originally Posted by fo0bar
ok, after a deeper dig it appears to be a little more complicated than i first anticipated.
i'm now on lcsd_merged_packing_slips.php - line 228. this appears to be where the magic happens. and it looks like i'm going to have to generate an array from the orders_product_attributes table each time the loop to display each product comes around.
If this is beyond the scope of the support here, it's all good - i understand it's not exactly superorders specific stuff that i'm trying to do now.
Thanks
[edit]
thanks - sorry, missed your reply. I think i've got it figured out now - or at least figured out how i'll do it. Thanks ayway!!
And now you see what I've said most everytime someone asks me about modifications to this packingslip.. It is not for the faint of heart.. (or at least it's beyond the scope of my abilities to mess with:smile:)Quote:
Originally Posted by fo0bar
It is beyond the scope of the support I can offer.. It was donated code, and a tad more complicated for me to tackle.. This doesn't mean someone from the Zen Cart community won't chime in here..:smile:Quote:
Originally Posted by fo0bar
Dunno what it is you are trying to accomplish.. It's possible that the updated packinglist might serve your needs.. That said I can make no promises on how soon I will finish and submit the new version.. (I'm working on it in my spare time..)
Does anyone have a solution for this, or is it a flawed feature and I'm better off giving up hoping that it'll ever work correctly?Quote:
Originally Posted by cycochuck
I also noticed that batch packing slips do not print the product attributes.
thanks - yeah, i think i've figured out how i'll do it. definitely not going to be simple. i understand what the code's doing, and how i'll pull the attribute array. Hopefully the shipping methods are done the same way, but i haven't had a look at that table structure yet.
just to fill you in, i'm selling a bunch of the "same" item, but slightly different specs. for example's sake, shirts. all the same size and style, but 15 different colours. I get a bunch of orders each day, and it's much more convenient to print a daily master packinglist - but i need the attribs/colour.
If anyone's interested and requires the same result, i'll post an update when i'm done. Hopefully it won't be too long.
Thanks for the help. I tried looking at the code to fix it myself but I'm not the best at programming and got lost.
Not a flawed feature at all.. Did you see the answer I posted to your question?? (1st post on this page..)Quote:
Originally Posted by cycochuck
There's nothing to "fix".. I posted the "solution"..Quote:
Originally Posted by cycochuck
I missed your post. But it really isn't an ideal solution. It is not saving much time having to right click on each frame and tell it to print frame. I might as well just go through super orders and click the invoice button on each order, especially when you have 15+ orders and the frames start becoming lines on the screen. To me, if it doesn't make doing something easier than the normal way it is either worthless or a flaw.Quote:
Originally Posted by DivaVocals
The All Invoices mod prints the invoices on separate pages but it only generates them based off of order status not date range. So, based off that mod, I know it is possible to generate say all invoices from April 1, 2011 to April 15, 2011 and just hit print and have them all come out on separate pages.
**deleted**
Then you are not understanding what I posted.. There is no flaw.. Both Firefox and Internet Explorer have always had options in the print command to print ALL frames on a page when printing.. (NO right clicking needed)
Firefox
http://i54.tinypic.com/2lc0gpi.png
Internet Explorer
http://i55.tinypic.com/9vf41e.png
Quote:
Originally Posted by cycochuck
The SO batch status options has MORE search options than the All Invoices mod. (Including DATE) And you CAN indeed print all invoices at once. (see my post above)Quote:
Originally Posted by cycochuck
And is that the most user friendly way to do it? The best way? Or is coding the actual result to do it for you the user friendly and best way?Quote:
Originally Posted by DivaVocals
First, you are forgetting that there are masses of people out there that barely tolerate computers and expect things to just work. Expecting them to know how and when to go into settings on print options to make this "feature" work as it implies only pisses them off. That then means I have to listen to a pissed off person because the "feature" does not work like it suppose to.
Second, with software trying to be user friendly, browsers have a habit of remembering your last print settings so all the user has to do is hit the print button. Your solution is requiring these same people that barely tolerate computers, to remember to double check their print settings every time they print.
I love super orders as a whole, it is that this part of it seems to be a rushed after thought of "that would be nice to add." It does not have the clean feel that the rest of the module has of "it does what it implies" and not require the user to change browser settings. Add in the fact that the batch packing lists don't show product attributes only shows to me, and the ones I have to listen to complain about it, that this is a flawed rush after thought.
I appreciate the great work that the developers have done making the module. But you have to look at this like a user, not a programmer. If the "feature" does not do what it says it'll do (or implies that it'll do) then the user sees this as a flaw. You can either go two routes when said users report a flaw. You can do the Microsoft route and call them stupid for not wanting to use it how you imagined how it would be used and refuse to fix it, or you can smack yourself in the head and say "they're right, I messed up."
Thanks Diva. That did the trick!Quote:
Originally Posted by DivaVocals
It's the way this mod has ALWAYS worked form the time it was first contributed to the community by Blindside over 4 years ago.Quote:
Originally Posted by cycochuck
MY solution?? Not MY solution at all.. SO is a solution that was GENEROUSLY contributed by a member of the Zen Cart community. He wrot it for HIS client (who paid for the development) and gave it back to the community.. My only role (and the role the folks who contributed to the updated version) was to update it to correct more SERIOUS issues than this one.Quote:
Originally Posted by cycochuck
It's not broken it merely doesn't function in a way you want it to. I'll admit that the documentation isn't clear on how to print batch invoices. Truthfully it never was clear, and when it was updated, we inadvertently forgot to include those instructions.
I haven't forgotten anything.. I work with computer challenged people everyday.. I have ALL my clients using SO and have instructed them how to print batch invoices with no issues whatsoever, I think it's MORE than possible to simply instruct someone how the batch print function works.
It's not flawed or rushed.. It simply wasn't included.. If you know the WHOLE history of this module, you know that it was a custom module that the original contributor wrote for HIS client. Please don't assume this was RUSHED, it's possible that Blindside's client did not want or need attributes on the batch invoices.. You would have to ask THEM why it wasn't included..Quote:
Originally Posted by cycochuck
I DO look at this like a user.. Because I work with USERS everyday.. That's why we made the changes and updates we did to Super Orders.. Please do not assume that because I don't necesarily think that what you are reporting is a "flaw" that we did anything less.. I simply don't agree with you that the batch invoice printing is a "flaw"Quote:
Originally Posted by cycochuck
and you might remember that this is not MICROSOFT.. This is a VOLUNTEER effort.. if you have a suggestion great.. make it and IF the VOLUNTEERS have time (or interest) they will take your suggestion into consideration.. Otherwise this is open source software.. there is NOTHING precluding YOU from making the suggested change yourself and contributing it back to the community.. This is how I got involved in the SO mod in the first place.. I was making changes to make it better based on feedback from MY clients.. If you would like to do the same for your clients, then there is nothing stopping you from updating the batch invoice printing to meet your client's needs..Quote:
Originally Posted by cycochuck
Thanks once again for a completely awesome contribution.
I've finished tinkering, and it turns out that with a bit of thought, my problem was much easier to solve than i initially expected - i now have attribute titles on the master pdf packinglist.
Here's the stupidly easy fix:
BACK UP THE FILE FIRST.
Open the following file:
[YOUR ADMIN FOLDER]/includes/functions/extra_functions/lcsd_merged_packing_slips.php
find this line:
REPLACE it with this:PHP Code:$orderText .= '(' . $order->products[$i]['qty'] . ') ' . $order->products[$i]['name'] . '/' . $order->products[$i]['model'];
And then the attribute title will show up after the model on the PDF Master Packing List.PHP Code:$attribText = (!$order->products[$i]['attributes'][0]['value']) ? '' : (' - ' . $order->products[$i]['attributes'][0]['value']) ;
$orderText .= '(' . $order->products[$i]['qty'] . ') ' . $order->products[$i]['name'] . '/' . $order->products[$i]['model'] . $attribText;
Hope this assists some people.
I've also done many other customisations to the packinglist pdf - a 'tick' box next to each item (with each item on its own line), shifted the order date and number to the right of the row, and also inserted the customer address. I'm working on postage type, but Austpost is making it difficult.
Let me know if anyone wants a copy of this customisation.
Just because that is the way its always been does not make it any less of a flaw. It just makes it a 4 year old flaw.Quote:
Originally Posted by DivaVocals
It does not have to be broken to be a flaw. It just has to not work the way the user expects.Quote:
Originally Posted by DivaVocals
I didn't come on to argue if this is a flaw or not. It is, even if you refuse to see. I just want to know if anyone had dealt with this issue or had any suggestion on how to fix this.Quote:
Originally Posted by DivaVocals
The attributes problem on the batch packing slips probably has been a flaw for 4 years as well until fo0bar gave the solution (thanks fo0bar.) So apparently there are flaws with the batch print forms.
If you're happy living with the flaw, fine. People live with flawed software everyday, look at Vista. I'm going to keep looking in the code to fix the flaw.
You're welcome..:smile:Quote:
Originally Posted by WriteAtHome
You are welcome.. Looks like you got your needs all worked out.. Gonna try and find time this weekend to get out the updates to Edit Orders and Super Orders.. The new Super Orders includes some improvements to the layout of the attributes on the PDF packingslips.. I'll repost the full changelog again later.. So you might wanna take a look at the improvements to the PDF packinglist..Quote:
Originally Posted by fo0bar
so because the way your user expects it to work does not line up with the way it ACTUALLY works, then it's a flaw?? I don't think so..Quote:
Originally Posted by cycochuck
It does make it a feature that neither you (based on your initial post where you inquired HOW to print batch invoices) or your user understood how to use. Part of the issue (again based on your initial post where you inquired HOW to print batch invoices) is that historically it's never been clearly explained how to print the batch invoice in the SO readme.. It has been the topic of several forum posts in the old support thread. Based on this I'll be sure that the next release include much clearer instructions..
Ummm okay... Neither did I.. I am free to disagree with you.. that IS still okay right??Quote:
Originally Posted by cycochuck
Ummmm... yeah.. you are aware that fo0bar and I are talking about the PDF packingslip (which was INTRODUCED to Super Orders in v3.0)..Quote:
Originally Posted by cycochuck
You can continue to call it a flaw if you like.. the fact of the matter this you came here to ask how to print the batch invoices.. You are not satisfied with the answer, and in YOUR opinion, you consider this a flaw.. For it to be a TRUE flaw, it wouldn't work at all.. It works, but HOW it works wasn't explained well, and YOU don't like the way it works..Quote:
Originally Posted by cycochuck
Awesome!!! Keep us posted if you come up with the required code changes..Quote:
Originally Posted by cycochuck
BTW, this is not entirely correct.. UNLESS your users are on another page with frames, using the frames printing settings wouldn't apply (eg: "browsers have a habit of remembering your last print settings" won't apply)Quote:
Originally Posted by cycochuck
1. LOVE this customization, thanks!!
2. I implemented this customization and it works, except for one little issue - it only displays the first attribute title for products that have multiple attributes. Suggestion to fix?
3. I would love to have your additional customizations to the PDF Packing List! Thank you for taking the time to do this and then sharing it with the community!
Quote:
Originally Posted by fo0bar
Support for multiple attributes are coming in the next Super Orders update.. The folks who so generously donated the PDF packingslip code (which is NOT... ahem.. FOUR YEARS OLD.. ahem..) were kind enough to make this fix.. This weekend might be when I get to finally submit this puppy.. :smile:Quote:
Originally Posted by ScriptJunkie
Awesome!
Quote:
Originally Posted by DivaVocals
Then they shouldn't be using a computer. That's like saying its acceptable for people to expect to drive a car just because a car is more convenient than walking, but that they shouldn't have to actually learn how to operate the vehicle.Quote:
Originally Posted by cycochuck
C'mon now. That's like saying a person should be able to click the "print" button in MS Word and expect it to print in duplex mode without having to activate that option. At some point, users have to be reasonable people if they want to get the most from their software.Quote:
Expecting them to know how and when to go into settings on print options to make this "feature" work as it implies only pisses them off.
Every job has its perks, eh? Seriously though, a little education for your users goes a long way, and when the updated ReadMe comes out with the next release - they'll have everything they need at their fingertips :)Quote:
That then means I have to listen to a pissed off person because the "feature" does not work like it suppose to.
Downloaded and installed and so far Super Orders is working great!
I did notice that the following files as posted in the instructions were NOT in included in the download:
super_orders_30/2_Required_Core_Edits/includes/languages/english/YOUR_TEMPLATE/account_history_info.php
super_orders_30/2_Required_Core_Edits/includes/modules/pages/account_history_info/header.php
super_orders_30/2_Required_Core_Edits/includes/templates/YOUR_TEMPLATE/templates/tpl_account_history_info_default.php
Should I be worried about this?
Kim
Not at all.. That's on me.. :blush: It was an oversight in the readme where I failed to explain correctly that those files were removed from v3.0. Those files were an an attempt in prior versions of SO to support hidden (admin only) comments. Hidden (admin only) comments is now a built in part of Zen Cart v1.3.9. So these files are obsolete and not included in the fileset.Quote:
Originally Posted by WriteAtHome
Glad you like SO.. Thank goodness for generous contributors like Blindside for creating it in the first place!!
Hi Fo0barQuote:
Originally Posted by fo0bar
As a fellow Aussie I am interested, especially if the customisations also suit OzPost.
Thanks for offering.
pm sentQuote:
Originally Posted by dw08gm
fo0bar - PLEASE feel free to share your changes here.. This way if you ever slip into the darkness:laugh:, folks won't be wondering where your cool changes went to..:smile:
As you can understand, I can't incorporate EVERYTHING people ask about here into the codebase.. From my perspective, Super Orders has to serve a WIDE audience.. So I make changes base don that perspective..
BUT...
others might be looking for the VERY change YOU are making and so I ENCOURAGE you to share your changes with the community!!
That said ScriptJunkie shared with me your changes to the "Master Packingslip List" (for those who don't know what this is, part of the PDF Packingslip features includes a master LIST of packing slips.. Think of it as a summarized order picklist)
My apologies for not understanding that the "Master Packingslip List" is what you were updating as opposed to the PDF Packingslip itself..
ANYWAY...
I'm LOVING the checkboxes and other changes you made to the "Master Packingslip List"..:clap: The addition of the customer address, and the modifications to the layout are outstanding and a must have IMHO!!! I have merged your changes into the "Master Packingslip List" in new fileset that I plan to submit (hopefully this weekend)
I am thinking that shipping method would also be something useful to include on the "Master Packingslip List" as well.. Gonna see if I can tackle that at some point for a later release..
I did not include your attributes code on the "Master Packingslip List" though.. Your code only supports ONE attribute (which may be FINE for some shopowners), and I think that for a wider audience, multiple attributes support will better serve a wider audience.
The upcoming update I plan to submit includes support for MULTIPLE attributes on the PDF packingslip, but NOT the "Master Packingslip List". (Oversight on my part for not seeing that when I BEGGED the original contributor to add multiple attribute support to the PDF Packingslip..:blush:)
I don't want to hold up the upcoming update any longer trying to get multiple attributes working on the "Master Packingslip List". I'm sure it's a simple adaptation of the new multiple attribute code from the PDF Packingslip into the "Master Packingslip List", but if you all saw the new code you'd know why a code challenged gal like me is a BIT hesitant to try adding it now..:laugh: I'm afraid it will further delay what has already been a long delayed update already.. (sorry I've been a LOT busy!!)
BUT that said I will give it the old college try to add multiple attribute support to the "Master Packingslip List" for another release..
Quote:
Originally Posted by fo0bar
....wow! i'm seriously honoured you're using my modifications!
my apologies to everyone about not sharing the code here :blush: i felt that since i'd mucked around so much, i'd lost track of exactly *where* i'd fiddled, and couldn't really give a "replace here to here" kind of submission :P
Also, advance apologies for my possible lack of forum-etiquitte...this is the first forum i've ever really comitted to. Complete forum noob here....educate me! corrupt me!
I don't mind putting in a little work if people find it useful - i'd be happy to have a bit of a hack at the multiple attribute titles issue - who knows if or when i might even need it! In theory it shouldn't be too hard - a bit of a loop around an array that already exists; i just haven't attempted it yet, as i thought this would be in the new version! And i'll make sure to post the mod here once i'm done this time :wink:
I've been continuing my work on this pdf output - i've also added the store logo to the top left, print date to top right, i'm working on a total item/value/packages count at the bottom, but *something* in the product loop keeps resetting the variables...I didn't want to go too far on it, as i thought this pdf output was what was being updated..didn't want to go reinventing the wheel!
Oh, and shipping method is definitely on of my top mods to work on. I need it. the AustPost shipping module is just making it a bit difficult for me to work with. I've got it displaying the "main" shipping method (for example, Australia Post), but not the 'sub' method (the choice between parcel/express).
Thanks again for an awesome contrib, i'm lovin' it.
Quote:
Originally Posted by DivaVocals
I can help with that :) I compared the file you gave me with the file that came with SO3. These are the resulting changes:Quote:
Originally Posted by fo0bar
1. Add tick/check box in front of each order item
2. Add spaces between each order item line
3. Decrease order item font size in order to make room for the addition of customer address
4. Moved Product ID Number and Order Date to align right
5. Added customer address (aligned right)
These changes make the master packing slip pdf (found in batch printing) much easier to read and much more functional! Thanks Fo0bar!
Here's the code:
EDIT 1
At or about Line 219, Find:
On the next line, Replace This:PHP Code:$pdf->Cell(10, 10, '', 'LRTB');
With This:PHP Code:$pdf->MultiCell(566, 12, $headingLine);
PHP Code:$pdf->SetFontSize('10');
$pdf->Cell(450, 12, $headingLine,'','0');
$orderRightBox = '#' . $order->info[id] . ' ' . zen_date_short($order->info['date_purchased']) . ' ';
$pdf->MultiCell('180','10',$orderRightBox,'','','R');
$pdf->SetFontSize('6');
$shipTo = zen_address_format($order->delivery['format_id'], $order->delivery, 0, '', "\n");
if($order->delivery['street_address'] == ''){
$shipTo = zen_address_format($order->customer['format_id'], $order->customer, 0, '', "\n");
}
$pdf->Cell('580','10',$shipTo,'','','R');
$pdf->MultiCell(566,2,'');
EDIT 2
At or about Line 226, Find:
On the next line, Replace This:PHP Code:if ($orderText != ''){
With This:PHP Code:$orderText .= ', ';
PHP Code:$orderText = '';
//$orderText .= ', ';
EDIT 3
At or about Line 227, Find:
On the next line, Replace This:PHP Code:}
With This:PHP Code:$orderText .= '(' . $order->products[$i]['qty'] . ') ' . $order->products[$i]['name'] . '/' . $order->products[$i]['model'];
PHP Code://This modification displays the attribute title in the line.
$attribText = (!$order->products[$i]['attributes'][0]['value']) ? '' : (' - ' . $order->products[$i]['attributes'][0]['value']) ;
$orderText .= ' ' . $order->products[$i]['qty'] . 'x ' . $order->products[$i]['name'] . '/' . $order->products[$i]['model'] . $attribText;
$pdf->Cell(27, 10, '');
$pdf->Cell(8, 8, '', 'LRTB');
$pdf->Cell(5, 0, '');
$pdf->SetFontSize('6');
$pdf->MultiCell(520, 10, $orderText); //display order text
EDIT 4
At or about Line 230, Find:
On the next line, Replace This:PHP Code:/* order object doesn't include orderid. added id to info so we can retrieve it here */
With This:PHP Code:$orderText = '[#' . $order->info[id] . ' ' . zen_date_short($order->info['date_purchased']) . '] ' . $orderText;
$pdf->Cell(27, 12, '');
$pdf->MultiCell(549, 12, $orderText);
PHP Code:$pdf->MultiCell(520,8,'');
Brilliant - all of you. Anyone for Cointreau with their coffee.
Have you seen thisQuote:
Originally Posted by fo0bar
http://www.zen-cart.com/forum/showpo...&postcount=212
Cheers
Excellent!
...and how far away is the new release? i've nailed the multiple attributes code if you'd like to include it, basically replace the code i mentioned above with the following code:
PHP Code:$attribCount=(sizeof($order->products[$i]['attributes']));
if($attribCount>1){
//more than one attribute in array
$aC=0;
while($aC < $attribCount){
$attribText .= ' - ' . $order->products[$i]['attributes'][$aC]['value'];
$aC++;
}
}else{
$attribText = (!$order->products[$i]['attributes'][0]['value']) ? '' : (' - ' . $order->products[$i]['attributes'][0]['value']) ;
}
$orderText .= ' ' . $order->products[$i]['qty'] . 'x ' . $order->products[$i]['name'] . '/' . $order->products[$i]['model'] . $attribText;
hope this helps :)
Quote:
Originally Posted by ScriptJunkie
Not sure I understand what "Total Qty vs Total Products" means.. A screenprint would help..Quote:
Originally Posted by dw08gm
The new release is as far away as the free time I have allows.. Sorry to be vague, but my day job and paid side work will ALWAYS trump free contributions.. :laugh::laugh:
Seriously though.. I hope to have it compiled and testing done this weekend.. All depends on my tester's schedule too.. I don't want to release this until I get feedback from testers.. Gonna try and toss something out tonight to the testers..
and BTW, I don't mind sharing code with others who will REALLY test and provide feedback.. But be prepared.. asking to test doesn't mean I will send out the code.. I tend to be VERY selective about who I share code with pre-submission because too many times I've had folks who didn't REALLY want to test, they simply wanted to get the "latest & greatest" version of a module.. (hideCategories is one example of where a few folks **ahem** stretched the truth about doing "testing" to scam .. I mean obtain a "pre-release" copy of the mod)
In the meantime, your multiple attributes code for the Master Packingslip List looks loads simpler than the multiple attributes code being used for the PDF Packingslip. I was planning on trying to incorporate the PDF Packingslip multiple attributes code into the Master Packingslip List, but yours looks MUCH simpler.. (and you say it's working?? hmmmm)
Quote:
Originally Posted by fo0bar
i've tested it with 2 attributes (only short term as it was on a live shop.. don't try this at home, kids), and it's working great; the way it's written should work for virtually unlimited attributes - it displays it as:Quote:
Originally Posted by DivaVocals
[product text]MODELNUMBER - att1 - att2 - att3
and all the single-attribute products display the same as in my previous mod :)
i'll be back in 12 hours - it's 4:30am!
[edit]
it's actually still running live with that mod; i haven't changed it back 'cause it's working - just removed the item's second attribute :P
I do not have a screenprint to hand howeverQuote:
Originally Posted by DivaVocals
Product A x 3
Product B x 2
Product C x 4
Total Products = 3 (A+B+C)
Total Quantity = 9 (3+2+4)
Provides extra QA. Useful when like products are bundled prior to final packaging, although can be problematic with products sold by dimension (eg metres/rope) or dollar amounts (eg donations, gift certs).
Hope this helps.
It's a good one, and VERY worthy of inclusion!!! Take a bow, you totally deserve it!!Quote:
Originally Posted by fo0bar
Ahhhh such a BIG OPENING... **sigh** I won't go there!!!:laugh:Quote:
Originally Posted by fo0bar
Let me make sure it's clear what code I was planning on submitting..:smile:Quote:
Originally Posted by fo0bar
What I have is the update to support multiple attributes on the PDF Packinglist, but NOT the PDF Master Packingslip LIST.
It was an over-sight on my part really not to ask the original contributor of the PDF Packingslip code to update the Master Packingslip List as well as the the PDF Packingslip.
BUT since you have provided a working version of the multiple attributes code for the Master Packingslip List, I add that and test it too..:smile:
Awesome.. Looking forward to seeing what you come up with.. Just remember that SHARESIES is always a GOOD thing!!:laugh:Quote:
Originally Posted by fo0bar
Ahhh.. Will have to have a looksee.. So the attributes are lined up horizontally next to the model number??Quote:
Originally Posted by fo0bar
The new multiple attributes code i have from the original contributor created arranges the attributes on the PDF Packingslip as follows:
Quote:
Product detail line
att1
att2
att3
yep - lined up horizontally - as my changes made the rows a bit higher, i tried to make each row take up as little height as possible - line returns equal paper wasted. that, and i'm O/C (not the lightswitch-flicking, doorknob-wiping variety though...more perfection and symmetry :P ) - single words on their own line just doesn't sit right with me, for whatever reason i don't really know.Quote:
Originally Posted by DivaVocals
feel free to play round with my bits of code, or if you think of something else to add to my list of mods to work on, let me know and i'll give it a shot!
Hey fo0bar,
I tested the multiple attributes support this afternoon, and it's not functioning as expected under all circumstances. While the circumstances under which it's gone wonky are not common, they could occur in a live store - and therefore the modifications must be able to handle them if they are to be included in a release of SO3.
Also, I discovered an issue with the modification that moves order numbers to the right.
Can you PM me your email address so I can send screen shots of the details? For whatever reason, I'm not able to post images to the forum successfully.
Diva wants to submit the newest SO3 update this weekend, so if the necessary changes are something you can make quickly, they can be included in the next release. If not, we'll hold off until the following release.
Lastly, the reason for needing vertical listing of multiple attributes (vs. the paper saving horizontal method) is that if there are long product and/or attribute names, they get truncated when formatted horizontally. Therefore, vertical formatting is necessary.
Having said that, if you personally prefer to use horizontal formatting, you should by all means do so. But is vertical formatting something you'd be able to work out for inclusion in the SO3 update release?
Quote:
Originally Posted by fo0bar
eep! i'm happy to continue working on it to make it a cover-all. i'll be PMing you my email address right away. I was *just* looking over my code and did notice that i hadn't done it *perfectly* to accommodate not having any attributes, but in my particular test it seems fine.Quote:
Originally Posted by ScriptJunkie
And yes - i'm absolutely more than happy to work out a bit of code to make the attribs display vertically - this isn't much work at all :D
Great! I'll go check my PM and email you right away :)
Quote:
Originally Posted by fo0bar
Quote:
Originally Posted by ScriptJunkie
Done. All issues fixed (including vertical attributes!), after basically rewriting the entire function. :frusty:
ScriptJunkie, I'll be emailing you a copy of the updated file so you can test it and see if it's functional and suitable for your cart. I think (read:hope) it'll be perfect for what you want. Diva, I'd send it to you but i don't have your email. PM me if you'd like me to email it to you.
for anyone else who wants to try this code:
1. BACK UP BACK UP BACK UP
2. open admin/includes/functions/extra_functions/lcsd_merged_packing_slips.php in a text editor.
3. find the following line:
select that line, and EVERYTHING UNTIL (but NOT including this line:PHP Code:function draw_packing_slip_master_list_customer($orderArray, &$pdf){
remember, do NOT select that line.PHP Code:function lcsd_merged_packingslips($selected_oids, $merge_selected_oids){
replace everything you've selected in the steps above, with this:
Again, I hope this helps.PHP Code:function draw_packing_slip_master_list_customer($orderArray, &$pdf){
$order = $orderArray[0];
$customerName = $order->delivery['name'];
if($order->delivery['street_address'] == ''){ /* this is to match the merged packingslips record */
$customerName = $order->customer['name'];
}
$totalQty = 0;
foreach($orderArray as $order){
for ($i=0, $n=sizeof($order->products); $i<$n; $i++) {
$totalQty = $totalQty + $order->products[$i]['qty'];
}
}
if ($totalQty == 1){
$headingLine = $customerName . ' (' . $totalQty . ' piece)';
}
else{
$headingLine = $customerName . ' (' . $totalQty . ' pieces)';
}
//sort out the header address line
$shipTo = zen_address_format($order->delivery['format_id'], $order->delivery, 0, '', "\n");
if($order->delivery['street_address'] == ''){
$shipTo = zen_address_format($order->customer['format_id'], $order->customer, 0, '', "\n");
}
$orderRightBox = $shipTo;
$pdf->SetFontSize('6');
$pdf->Cell('580','10',$orderRightBox,'','','R');
$pdf->Line(18,$pdf->GetY(),594,$pdf->GetY());
$pdf->Ln(2);
$pdf->SetFontSize('10');
$pdf->Cell(10, 10, '', 'LRTB');
$pdf->Cell(200, 12, $headingLine,'','0');
$pdf->Ln(10);
$pdf->MultiCell(566,2,'');
foreach($orderArray as $order){
$orderText = '';
for ($i=0, $n=sizeof($order->products); $i<$n; $i++) {
if ($orderText != ''){
$orderText = '';
}
$pdf->Ln(4);
$pdf->Cell(27, 10, '');
$pdf->Cell(8, 8, '', 'LRTB');
$pdf->Cell(5, 0, '');
$pdf->SetFontSize('6');
$multiAtt = false;
if(sizeof($order->products[$i]['attributes']) >0){
$attribCount=(sizeof($order->products[$i]['attributes']));
if($attribCount>1){
$aC=0;
while($aC < $attribCount){
$attribText = ' - ' . $order->products[$i]['attributes'][$aC]['value'];
if($aC==0){
$orderText .= ' ' . $order->products[$i]['qty'] . 'x ' . $order->products[$i]['name'] . '/' . $order->products[$i]['model'] . $attribText;
$pdf->Cell(380,10,$orderText,0,0);
$dateId = zen_date_short($order->info['date_purchased']) . ' #' . $order->info[id];
$pdf->SetFontSize('10');
$pdf->Cell('140','10',$dateId,'0','1','R');
$multiAtt=true;
$pdf->SetFontSize('6');
}else{
$widthTest = (' ' . $order->products[$i]['qty'] . 'x ' . $order->products[$i]['name'] . '/' . $order->products[$i]['model']);
$spaceWidth = $pdf->GetStringWidth($widthTest) + 40;
$orderText = $attribText;
$pdf->Ln(0);
$pdf->Cell($spaceWidth,10,'');
$pdf->Cell(20,12,$orderText,0,0);
}
$aC++;
}
}else{
$attribText = (!$order->products[$i]['attributes'][0]['value']) ? '' : (' - ' . $order->products[$i]['attributes'][0]['value']) ;
$orderText .= ' ' . $order->products[$i]['qty'] . 'x ' . $order->products[$i]['name'] . '/' . $order->products[$i]['model'] . $attribText;
$pdf->Cell(380,10,$orderText);
}
}else{
$orderText .= ' ' . $order->products[$i]['qty'] . 'x ' . $order->products[$i]['name'] . '/' . $order->products[$i]['model'];
$pdf->Cell(380,10,$orderText);
}
$attribText = '' ;
$orderIdText = $order->info[id];
if ($orderIdText == $previousOrderId){
$pdf->MultiCell('580','10',' ','','','R');
}else{
if (!$multiAtt){
$pdf->SetFontSize('10');
$dateId = zen_date_short($order->info['date_purchased']) . ' #' . $order->info[id];
$pdf->Cell('140','10',$dateId,'0','1','R');
$pdf->Ln(0);
$previousOrderId = $order->info[id];
}
}
}
$pdf->Ln(8);
}
}
Hi Diva
Any chance of you adding a heading and dashed underline immediately above the customer comments of the email output in the next release.
eg line 226 and elsewhere in admin\includes\functions\extra_functions\common_orders_functions.php
TABLE_HEADING_COMMENTS . "\n" . EMAIL_SEPARATOR . "\n" . strip_tags($notify_comments) .
Thanks in advance
The upcoming release is going to be eliminating the need for this function file at all as I have "synchronized" and "reconciled" the comments code from Super Orders with the default comments code used in orders.php. (doing the same for Edit Orders as well)
Doing so will make any future changes with comments in Zen Cart easier to incorporate into Super Orders (and Edit Orders)
Quote:
Originally Posted by dw08gm
and so I'm CLEAR, these are changes to the PDF "Master list of packing slips" correct??:smile:
Quote:
Originally Posted by fo0bar
Quote:
Originally Posted by DivaVocals
Correct :)
I'll be beginning testing in the next few minutes :)
you got it!Quote:
Originally Posted by DivaVocals
...i'm pretty keen to know how it tests - i'm slightly limited when it comes to testing this multiple attribute stuff in a live store :P