search

quickapps / cms

Modular CMS powered by CakePHP
GNU General Public License v3.0
164 stars 69 forks source link

List type fields #168

Closed paulrwest closed 7 years ago

paulrwest commented 7 years ago

The instructions for List Type options states:

The label is optional: if a line contains a single string, it will be used as key and label.

I found that this is not true if you only use one string null is saved in the value if you use the key|label format it works correctly

botchris commented 7 years ago

Are you using Windows? Seems like there is a problem related to carriage return symbol on Windows ("\n\r") OS

paulrwest commented 7 years ago

yes I am on windows

On Wed, Nov 2, 2016 at 9:04 PM, Christopher Castro <notifications@github.com

wrote:

Are you using Windows? Seems like there is a problem related to carriage return symbol on Windows ("\n\r") OS

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/quickapps/cms/issues/168#issuecomment-257998720, or mute the thread https://github.com/notifications/unsubscribe-auth/APGVWqdpz7d6PKMhvx1fxaynbHjzE9dEks5q6Pr6gaJpZM4KnZI1 .

botchris commented 7 years ago

please confirm this patch 64c200a326daff57ab7eeec6badf9336358d30f1 solves the problem.

paulrwest commented 7 years ago

Yes solved! thank you