Open balinterdi opened 6 years ago
Maybe I'm wrong but if the value of the second parameter passed to moment-format is a falsy value, I'd expect the date to be displayed in the default format.
moment-format
That's currently not the case as the code checks the number of parameters passed explicitly and only uses moment.outputFormat (the default format set in the configuration): https://github.com/stefanpenner/ember-moment/blob/v7.7.0/addon/helpers/moment-format.js#L28-L35
moment.outputFormat
If it worked as I describe, the following:
{{#with currentUser.user.dateFormat as |format|}} {{moment-format date format}} {{else}} {{moment-format date}} {{/with}}
could be written as:
{{moment-format date format}}
Is this the intended behavior or can we get the default format in the case I described?
Maybe I'm wrong but if the value of the second parameter passed to
moment-format
is a falsy value, I'd expect the date to be displayed in the default format.That's currently not the case as the code checks the number of parameters passed explicitly and only uses
moment.outputFormat
(the default format set in the configuration): https://github.com/stefanpenner/ember-moment/blob/v7.7.0/addon/helpers/moment-format.js#L28-L35If it worked as I describe, the following:
could be written as:
Is this the intended behavior or can we get the default format in the case I described?