Comments on: My First Foray into Developing on a Web Platform for Fun and (non)-Profit http://absolutgcs.org/2008/10/08/my-first-foray-into-developing-on-a-web-platform-for-fun-and-non-profit/ thoughts and observations Fri, 05 Feb 2010 09:47:58 -0800 http://wordpress.org/?v=2.9.2 hourly 1 By: absolutgcs http://absolutgcs.org/2008/10/08/my-first-foray-into-developing-on-a-web-platform-for-fun-and-non-profit/comment-page-1/#comment-46 absolutgcs Mon, 13 Oct 2008 01:23:22 +0000 http://absolutgcs.org/?p=82#comment-46 I believe the regex permits what ICANN passes as valid email addresses. & isn't permitted in email addresses. Given that this particular check has been tested millions of times on wordpress.com and hosted wordpress.org blogs, I'm guessing it's working. I have a default delimiter set that already covers the majority of cases, but in case customization is needed, as requested by the customer, it is there as a feature. You need to remember that schools work with a variety of different systems with quite a bit of legacy so I can understand why they might have this need. I believe the regex permits what ICANN passes as valid email addresses. & isn’t permitted in email addresses. Given that this particular check has been tested millions of times on wordpress.com and hosted wordpress.org blogs, I’m guessing it’s working.

I have a default delimiter set that already covers the majority of cases, but in case customization is needed, as requested by the customer, it is there as a feature. You need to remember that schools work with a variety of different systems with quite a bit of legacy so I can understand why they might have this need.

]]> By: Don http://absolutgcs.org/2008/10/08/my-first-foray-into-developing-on-a-web-platform-for-fun-and-non-profit/comment-page-1/#comment-27 Don Thu, 09 Oct 2008 06:03:31 +0000 http://absolutgcs.org/?p=82#comment-27 Hmm, unless I'm reading the regex wrong (which is very likely, I've always had another developer do the regex stuff), your pattern doesn't allow for '&'s. You should add an allowance for that and some other characters I can't remember off the top of my head. That's still interesting that the delimiters need to be said. Regardless of the source, it would still seem that having commas, semi-colons and spaces be used as default delimiters would be pretty robust and fulfill 90+% of your needs. You could add tabs and pipes (|) to that collection and that should cover 99%. Not to belabor the point, it's just that I like to see less options, especially when the solution seems easy enough to implement. I can think of another good plugin you might want to build for WordPress, we should discuss it sometime. :) Hmm, unless I’m reading the regex wrong (which is very likely, I’ve always had another developer do the regex stuff), your pattern doesn’t allow for ‘&’s. You should add an allowance for that and some other characters I can’t remember off the top of my head.

That’s still interesting that the delimiters need to be said. Regardless of the source, it would still seem that having commas, semi-colons and spaces be used as default delimiters would be pretty robust and fulfill 90+% of your needs. You could add tabs and pipes (|) to that collection and that should cover 99%. Not to belabor the point, it’s just that I like to see less options, especially when the solution seems easy enough to implement.

I can think of another good plugin you might want to build for WordPress, we should discuss it sometime. :)

]]> By: absolutgcs http://absolutgcs.org/2008/10/08/my-first-foray-into-developing-on-a-web-platform-for-fun-and-non-profit/comment-page-1/#comment-26 absolutgcs Wed, 08 Oct 2008 21:55:28 +0000 http://absolutgcs.org/?p=82#comment-26 Regarding the delimiters, I forgot to mention this was a requirement from the customer. Like you mentioned there are common delimiters that are used but this was intended to be configurable because the list of addresses could come from a variety of sources. The email address regex string came from the is_email function in Wordpress: <code> $chars = "/^([a-z0-9+_]|\\-|\\.)+@(([a-z0-9_]|\\-)+\\.)+[a-z]{2,6}\$/i"; </code> As far as the plugin name, well, I don't think it's much of a management system haha. We'll see what else comes of it =) Regarding the delimiters, I forgot to mention this was a requirement from the customer. Like you mentioned there are common delimiters that are used but this was intended to be configurable because the list of addresses could come from a variety of sources.

The email address regex string came from the is_email function in Wordpress:

$chars = "/^([a-z0-9+_]|\\-|\\.)+@(([a-z0-9_]|\\-)+\\.)+[a-z]{2,6}\$/i";

As far as the plugin name, well, I don’t think it’s much of a management system haha. We’ll see what else comes of it =)

]]> By: Don http://absolutgcs.org/2008/10/08/my-first-foray-into-developing-on-a-web-platform-for-fun-and-non-profit/comment-page-1/#comment-25 Don Wed, 08 Oct 2008 10:43:40 +0000 http://absolutgcs.org/?p=82#comment-25 Just notes as I read your post... - The delimiters shouldn't need to be defined. Commas, spaces, and semi-colons are standard delimiters, your regex should just check for those delimiters by default, allowing the user to mix in any of them as a delimiter. - I'm curious as to what your email address regex looked like. I've found that the regexes "published" on websites typically don't contain characters like '&', etc. We've had to modify our pattern a ton of times to get it just right. - You might want to change the name of the plugin from "Bulk User Add" to "Giri's Uber (but first) User Management System of User Managings". =) Just notes as I read your post…

- The delimiters shouldn’t need to be defined. Commas, spaces, and semi-colons are standard delimiters, your regex should just check for those delimiters by default, allowing the user to mix in any of them as a delimiter.

- I’m curious as to what your email address regex looked like. I’ve found that the regexes “published” on websites typically don’t contain characters like ‘&’, etc. We’ve had to modify our pattern a ton of times to get it just right.

- You might want to change the name of the plugin from “Bulk User Add” to “Giri’s Uber (but first) User Management System of User Managings”. =)

]]>