SUDatePicker >> formattedResult relies on Pharo custom #printFormat

SeasideSt / Seaside

The framework for developing sophisticated web applications in Smalltalk.

MIT License

518 stars 71 forks source link

SUDatePicker >> formattedResult relies on Pharo custom #printFormat #1265

Open marianopeck opened 3 years ago

marianopeck commented 3 years ago

Such method is coupled to Pharo's Date printing/formatting which is not available in other Smalltalk dialects. Not sure how to fix this as I am not sure Grease offers a solution.

jbrichau commented 3 years ago

Looking at this issue, I noticed that even in Pharo, the format is no longer used... it delegates to BasicDatePrinter which drops the format argument without doing anything with it. What would be produced is an iso8601 date.

Furthermore, the SUDatePicker component is broken. I never used it, there are no tests for it and it's terribly outdated. I'm just going to remove Scriptaculous from Seaside alltogether.

So we can ignore this.

marianopeck commented 3 years ago

I think I agree. I don't know anyone using Scriptaculous and I doubt it still makes sense to keep the Seaside integration. So..I am fine removing it alltogether