missing line end when export product description

[chaddro]

missing line end when export product description

[mc12345678]

So, the function was added when similar code was seen throughout the existing program and to allow similar changes as suggested above to be applied throughout rather than against one specific use or another. I can not speak to the original design of removing the various returns other than it apparently did cause problems for editing the file(s) and therefore was added at that time. I would say that generally speaking such a "return" probably should include the html <br /> or <br> tag which could be applied using the php code nl2br($string) to insert the first of those two before the hard coded "return".

Whether I added the needs review statement or it existed previously, I would say that if I did so it was because it didn't cause me any problems as I use html tags in the content and therefore such "single paragraph" result isn't a problem, but what isn't a problem for me might be a problem for someone else, therefore recognition that perhaps some review would provide some alternate solutions applicable to desired situations. Certainly, one is able to make the changes they see fit once the software is downloaded. :)

As commented, it is something to consider; however, because not everyone uses Open Office as suggested, the issue may have to be addressed in some other way or remain a minor issue at least until other "desirable" code is suggested.

[xiezhensheng]

I see the dilemma here, and different use cases.

I think switch to html based product description is a good idea, not a fix, but a bypass for good reasons.

For now, I am going to leave only \t as the problematic char and to remove when export. As my test, keeping \r and \n in the product description won't break anything. However if I leave \t unfiltered, the export will be broken. To lazy to investigate why...

[mc12345678]

Understand the "too lazy to investigate" part. :)

As to using one method for export and possibly the same or another for import, have you worked out how those two different directions will act differently if at all?

In response to this I focused primarily on export and am glad you updated the original post. The quoted code only related to SBA and that made the situation confusing but above as provided the situation is clearer.

[xiezhensheng]

The "clear up" code is every where and confusing :) I feel in some of the case there is double "clear up" situation. I think at some point the code needs to be cleared.

I don't have time/resources to test both directions. I feel it is more risky to filter less "problematic words", but this is convenient when I don't want to use complex html editing.

However, to make my website looks pro, I guess I will eventually switch to html based product description.