Thread Rating:
  • 0 Vote(s) - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
Internal Stock Request - Many Issues
02-13-2018, 10:26 AM,
#21
RE: Internal Stock Request - Many Issues
Changes committed to SVN.

Paul, that should cover all the items mentioned in the first post to this thread.
Reply
02-13-2018, 10:37 PM, (This post was last modified: 02-13-2018, 11:17 PM by VortecCPI.)
#22
RE: Internal Stock Request - Many Issues
Thank you so much Paul!
I am such a PHP novice... I had no idea a comma was preferred for printing strings...
https://www.linkedin.com/in/eclipsepaulbecker
Reply
02-14-2018, 03:05 AM, (This post was last modified: 02-14-2018, 04:14 AM by VortecCPI.)
#23
Information  RE: Internal Stock Request - Many Issues
I am getting a "The number format is wrong" error on the Lot/Batch/Serial text field.

We have hyphens in our Lot/Batch/Serial Numbers so I assume the class="number" within the INPUT tags is no good.
Also...

Other input forms have MaxLength set so we should likely do the same here.

   

Attached is my attempt to make code uniform with the other serial entry script.


Attached Files
.php   InternalStockRequestFulfill.php (Size: 16.46 KB / Downloads: 1)
https://www.linkedin.com/in/eclipsepaulbecker
Reply
02-14-2018, 08:29 AM, (This post was last modified: 02-14-2018, 08:58 AM by TurboPT.)
#24
RE: Internal Stock Request - Many Issues
(02-13-2018, 10:37 PM)VortecCPI Wrote: I am such a PHP novice... I had no idea a comma was preferred for printing strings...
I don't know that commas a preferred, per say, as I've been more accustom to the concatenated way with the period. This has come from working with weberp, believe it or not.

I (very recently) first noticed the comma usage with some of Tim's code, and (not immediately noticed) there is mixed use of the two ways within some weberp code areas. With one of the files from Tim, I left those as-is when dropped in, and since the Internal Stock file was getting many changes, I went ahead and converted the concat to commas in the echo's.

For me, I will try to use commas going forward, as I can see there can be a slight performance over the concatenation. However, I do not plan on applying such a change to larger files, but will do so bit-by-bit during edits. Caution has to be taken anyway, for the concatenation in SQL strings must use the period and not a comma, so the change would only be relevant with echo output.
Reply
02-14-2018, 08:53 AM, (This post was last modified: 02-14-2018, 09:01 AM by TurboPT.)
#25
RE: Internal Stock Request - Many Issues
(02-14-2018, 03:05 AM)VortecCPI Wrote: I am getting a "The number format is wrong" error on the Lot/Batch/Serial text field.

We have hyphens in our Lot/Batch/Serial Numbers so I assume the class="number" within the INPUT tags is no good.
Also...
Yes, the 'number' can potentially have a dual purpose...

There is CSS to right-align the text when class="number".

As you've discovered, when class="number" is applied to inputs has javascript handling applied as well, to help "filter" the input. (MiscFunctions.js, lines 368, 375 -- and the function to line 375 up at line 13)

However, looking at that code now, it seems that hyphens should be allowed, based on line 20? I'll have to double-check.

===============

So based on post #23, you were only having trouble with the attached file?
Reply
02-14-2018, 09:35 AM,
#26
RE: Internal Stock Request - Many Issues
I had trouble with the file in SVN but the file attached in post #23 works for my needs.

It just needs a sanity check by somebody more qualified like you!
https://www.linkedin.com/in/eclipsepaulbecker
Reply
02-14-2018, 09:55 AM, (This post was last modified: 02-14-2018, 12:27 PM by TurboPT.)
#27
RE: Internal Stock Request - Many Issues
...trouble with the file in SVN??

Any other trouble beyond what you've already described to know?
Reply
02-14-2018, 10:35 AM,
#28
RE: Internal Stock Request - Many Issues
Nope. That was the only issue I ran into.
https://www.linkedin.com/in/eclipsepaulbecker
Reply
02-14-2018, 11:17 AM, (This post was last modified: 02-14-2018, 11:41 AM by TurboPT.)
#29
RE: Internal Stock Request - Many Issues
Ok, I see what you mean.

A hyphen CAN be there, but only if leading, as in a negative value. Anywhere else gives the 'bad number format' message.
So yes, in that particular case, "number" would not be good to apply to that specific input.

I also double checked the stockserialmoves table... column serialno is a varchar type, so that change will be fine.

However, one minor tweak though: I will increase the maxlength to match the column limit (30), but 'size' will remain the same.

Committing in just a moment...
Reply
02-14-2018, 11:48 AM,
#30
RE: Internal Stock Request - Many Issues
Thank you so much Paul! We will be using this piece a lot so all your help is greatly appreciated!
https://www.linkedin.com/in/eclipsepaulbecker
Reply


Forum Jump:


Users browsing this thread: 2 Guest(s)