phpBMS

Ticket #16 (closed enhancement: fixed)

Opened 5 years ago

Last modified 22 months ago

Client/Order State Field not long enough

Reported by: brieb Owned by: brieb
Priority: minor Milestone: 0.7
Component: phpbms Version: 0.61
Keywords: Cc:

Description (last modified by brieb) (diff)

Some countries use more than a two letter abbreviation for state/province/region. We should increase this to 5 chars

Change History

Changed 5 years ago by brieb

  • keywords review added

I'm also wondering if we should change the state to an autofill, or drop down list of some sort. Comments or thoughts?

Changed 5 years ago by brieb

  • type changed from defect to enhancement
  • milestone changed from unknown to 0.62

Changed 5 years ago by brieb

  • status changed from new to assigned

Changed 5 years ago by brieb

  • milestone changed from 0.62 to 0.70

Changed 5 years ago by brieb

  • description modified (diff)
  • summary changed from Client/Order State Fileds not long enough to Client/Order State Field not long enough

Changed 5 years ago by brieb

  • keywords review removed
  • status changed from assigned to closed
  • resolution set to fixed

Implemented by [178]. Went with a length of 5, as no other input was given.

Changed 5 years ago by jayson.barker@…

Hello,

I am aware you have closed this ticket, however i feel that a length of 5 is still too short, In the UK for example there are many counties that have far longer names, it is quite important to include the full name of the county as we do not abbreviate.

I would propose a limit of 15-20, as there are counties with names like Buckinghamshire,Northamptonshire,West Midlands.

Im not sure what other counties are like for province names.

Regards Jay

Changed 5 years ago by anonymous

  • status changed from closed to reopened
  • resolution fixed deleted

Changed 5 years ago by brieb

  • status changed from reopened to closed
  • resolution set to fixed

We are open to this idea. However, why don't you open a new ticket (like extending from 5 - 15 to 20) so that we can include this one in the upcoming release of 0.7

Also, you can assign the task to yourself, and submit a patch with the necessary changes to extend it to 15 characters. Use [178] as a guide of what/where it needs to changed in order to support this. Hopefully we can get this change in for milestone 0.8

Changed 4 years ago by anonymous

  • status changed from closed to reopened
  • type changed from enhancement to defect
  • version changed from 0.61 to 0.7
  • resolution fixed deleted

I am using v0.7 but cannot seem to get more than 5 characters. I have used the guide in [178] and everything is set as it should be, even when maxlength="5" changed to 20 in invoices_addedit.php and clients_addedit.php it still wont let me have more than 5 characters.

Any help much appreciated.

Changed 4 years ago by brieb

  • status changed from reopened to closed
  • type changed from defect to enhancement
  • version changed from 0.7 to 0.61
  • resolution set to fixed

If you notice , the change also alters the database in addition to changing the HTML field lengths.

Also the best place to get help is to use the forum.

Changed 2 years ago by anonymous

I am aware you have closed this ticket, however i feel that a length of 5 is still too short, In the UK for example there are many counties that have far longer names, it is quite important to include the full name of the county as we do not abbreviate.

I would propose a limit of 15-20, as there are counties with names like Buckinghamshire,Northamptonshire,West Midlands. nike air yeezy shoes

Im not sure what other counties are like for province names.

Note: See TracTickets for help on using tickets.
Scanned by Orvant Copyright © 2010 Kreotek, LLC. All Rights reserved.