align toString and fromString for OpenLayers.Bounds

[bartvde]

Erik's reply: hi bart, the toString() function I think was mostly just designed for debugging... not so much as a way to translate the bounds into a storable string format. I think your point is a good one... unfortunately, it wouldn't be proper of us to change a thing like this now because it would be a breach of API. for the OpenLayers 3.0 release, however, I think this would be a great thing to change. erik ps. if you really need this functionality, we could put in a patch for a function with a different name (saveToString() or something like that) for the time being and mark the current toString() as deprecated...

Original email: Hmm, one more thing, isn't it strange that toString and fromString use different string formats?

I was hoping to use toString and create a bounds again from the same string using fromString.

But toString uses: left-bottom=(23560.672130601146,384199.8282873455) right-top=(50198.73747014423,401559.87359025853 )

and fromString interprets only something like: 23560.672130601146,384199.8282873455,50198.73747014423,401559.87359025853

[tschaub]

Is there consensus that changing the format of the string that toString returns is a change we can't make before 3.0?

I wouldn't be opposed to changing the format of the string.

[tschaub]

Another note - the more things we can squeeze into 2.x, the easier it will be for people to migrate to 3.0.

[crschmidt]

I don't think that changing the format of the toString output is a violation of the API. I'd be willing to take this in a 2.x.

[adube]

patch changing the 'toString' method. The old one is renamed 'toHRString'.

[adube]

I agree that the method has no real purpose whatsoever and changing it as mentioned above wouldn't hurt anything. Attached (above) a patch fixing the issue. I'd say this ticket is ready for review.

[erilem]

Replying to crschmidt:

I don't think that changing the format of the toString output is a violation of the API. I'd be willing to take this in a 2.x.

+1

[fredj]

change toString without keeping old implementation. update unit tests

[fredj]

(In [11755]) align toString and fromString methods. r=erilem (closes #1275)