Hi Ciaran,
Glad to hear your having some successes with the plugin.
The bugs you have highlighted have already been put forward in other threads and emails:
http://wordpress.org/support/topic/pdf-and-admin-column-width
The currency sign is going in because of the decimal. I have it set to determine a price by decimal, which now that a number of people have had the same issue as you, I see that isn’t the best way to do it.
I’m currently in development of some major changes to the plugin, which includes a better way of defining the type of field your putting in a column (ie setting it to text, numeric, price, etc) which will then handle the text wrapping and the currency alot better.
The unfortunate news of this is that I have been forced to put this on the back burner whilst I have freelance work to complete.
I’ll keep you updated about the progress of these fixes.
Thanks again
Ash
Hi there Ash,
Thank you very much for taking the time to respond with great detail in what is causing the issue. I did a quick search first and didn’t see the threads relating to this being asked. Thank you for pointing to the right thread for me to read.
I 100% understand the need to put free work aside whilst you take care of paid freelance work. Family first, interests, hobbies and free development last.
The major changes that you are working on sounds exciting. If this was in a repo, I could assist with some minor things, and would be happy to do so.
Ciaran
Hey Ciaran,
As i’m sure you have seen, update has been released with some new options, one being the better column control which allows you to choose the column type. Selecting ‘text’ will enable the column to wrap.
Cheers mate
Thank you Ash, I will update the plugin that is on the site now. Good timing for this since the site will go live next week for the multisite function for having sub contractors being able to create their invoices. So far all testing has proved very positive in making this into an invoicing machine in a way.
Hi Ash,
Getting some errors now since the latest update. Not sure what they are from. I have added the column widths as 25%, added some more columns, then changed the widths to 15%.
Warning: Missing argument 2 for on_update_invkp_column_widths() in /home/chome/public_html/wp-content/plugins/invoice-king-pro/includes/admin_area.php on line 1129
Warning: Cannot modify header information - headers already sent by (output started at /home/chome/public_html/wp-content/plugins/invoice-king-pro/includes/admin_area.php:1129) in /home/chome/public_html/wp-includes/pluggable.php on line 875
Hopefully this helps.
Ciaran
Ash,
even with this error showing, the plugin still seems to work. It only shows the error when I press save settings.
Ciaran
Hey Ciaran,
Just got back online (Melbourne Cup Day here in Aus). This is priority on my list. I’ll have an update for you in a few 😉
Cheers for the review too
Thanks
Ash
Ash, I won for the first time mate. $48 on #6. $5 each way, and was one horse short of getting the trifector as well. DOH. Adelaide boy here..
Ciaran
Hey mate,
Ah, Sydney here 😉 And seems your a 24 hour worker like me. Just assumed you were elsewhere due to some of the late night replies.
I always pick the early runners. Had Brown Panther – as soon as it popped up in the top for out of the gates, I knew it was a no hoper. Gambling isn’t for me, I’ll stick to development 😉
Ciaran,
As the above errors are only warnings (which is why it continues to function correctly), I will hold off releasing an update just for that. I believe the warnings are just due to there not being an old value and i’m not using it either so I will omit it.
Cheers
Morning Ash,
I can let you know what started the issue. Yesterday the client came back and advised to add 3 new table (text) fields. I added them, then realised i could not re-sort the fields the way I wanted (was hoping to drag them into the new order). I then removed the values in the fields, and placed the values in the fields the way I needed them to be ordered.
I probably should have deleted the old fields first, then created the new ones.
Hopefully this helps to figure out what I have done to replicate the error warnings.
Hey Ciaran,
I believe I was able to replicate your issue. It would seem I forgot to update the js file to include the width field when you add a field.
Theres a new version, 1.1.1, which should see your error fixed.
Thanks
Ash
