https://bugs.freedesktop.org/show_bug.cgi?id=67763
--- Comment #8 from Lionel Elie Mamane <[email protected]> --- (In reply to comment #6) > (In reply to comment #5) > > Started with the wrt2-branch. > Working my way through the wrt-2 branch, I notice that > offapi/com/sun/star/text/TimeDisplayFormat.idl is marked af deprecated. Are > you sure that is the right start-point? Thx for yout input@ At first, I rather suspected that the "deprecated" flag is in error, since it was marked as deprecated when it was introduced (14 Nov 2000, commit ID 20b69e51231b64c9c595ae40dc2c2bb76a8c4e0e). But then, this constant group is used only in service ::com::sun::star::text::textfield::DateTime, where the property where it is deprecated anyway. OTOH, concretely the choices in offapi/com/sun/star/text/TimeDisplayFormat.idl are the choices one has in the GUI when choosing the format of a time field or a date field. So - to me - they seem to be in active use within LibreOffice proper, which is rather incompatible with deprecation. Unless I'm wrong and t Also, the listed alternative (using a NumberFormatter) means shoe-horning a date / time in a double, which leads to the problems listed in comment 0. Grr... I guess we need to look at this very carefully and maybe un-deprecate it. Ah no... I think I was just wrong... I thought the list of formats of a date/time control was a value of ::com::sun::star::text::textfield::DateTime, but actually it seems to be listed in forms/source/misc/limitedformats.cxx, function lcl_getFormatTable. So possibly that's the right starting point. But (same file) function OLimitedFormats::ensureTableInitialized, the description is subject to a lookup by "xStandardFormats->queryKey", so (I guess) these "Standard Formats" will have to be extended. Need to find out where they are, though. I guess this corresponds to the "FormatCode" in i18npool/source/localedata/data/*.xml, which would need update for all languages (which in hindsight seems natural). Obviously, those locales that you don't know will be "done" by our i18n teams. (In reply to comment #7) > (In reply to comment #6) > > I notice that > > offapi/com/sun/star/text/TimeDisplayFormat.idl is marked af deprecated. Are > > you sure that is the right start-point? > > Is now suspect TimeDisplayFormat.hdl is the way to go. That's where I'm > going now. TimeDisplayFormat.hdl is generated automatically from TimeDisplayFormat.idl -- You are receiving this mail because: You are on the CC list for the bug.
_______________________________________________ LibreOffice mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libreoffice
