9 Dec
2011
9 Dec
'11
10:17 a.m.
Hi Devs,
I am looking at now using the new Locale added in DocumentReference into
the implementation of XWikiDocument.
I have already deprecated language related stuff in XWikiDocument, and I
have introduce a XWikiDocument#getLocale and an XWikiDocument#isTranslation
helper since the deprecation of defaultLanguage will increase the need of
it. I have also added XWikiDocument#getTranslatedDocument() with Locale in
place of language. All the changes are almost backward compatible, which is
nice (there is some subtleties with default, "" and null that is now more
equivalent, but should not have consequences).
The is however one change that is not backward compatible, which is the
change of the document reference. Therefore,
XWikiDocument#getDocumentReference does not return the same reference than
it does before, because this reference now contains the Locale. This cause
breakage in several places. I see some option to fix this:
A. Fix all places broken. This may be too long for me, and not trivial.
B. Introduce a new XWikiDocument#getDocumentReferenceWithLocale() and
have XWikiDocument#getDocumentRefence() returns without Locale. Very easy,
but not nice.
C. Introduce a new XWikiDocument#getDocumentReferenceWithoutLocale() and
change all existing calls to XWikiDocument#getDocumentRefence() in platform
to use this one. Nicer, but this is not fully backward compatible.
Since I am on it right now, I would appreciate your opinion quickly.
WDYT ?
I am undecided between B and C
--
Denis Gervalle
SOFTEC sa - CEO
eGuilde sarl - CTO
9 Dec
9 Dec
1:41 p.m.
New subject: [xwiki-devs] [VOTE] Adding the Locale into the document reference of XWikiDocuments
On Fri, Dec 9, 2011 at 10:17, Denis Gervalle <dgl(a)softec.lu> wrote:
Hi Devs,
I am looking at now using the new Locale added in DocumentReference into
the implementation of XWikiDocument.
I have already deprecated language related stuff in XWikiDocument, and I
have introduce a XWikiDocument#getLocale and an XWikiDocument#isTranslation
helper since the deprecation of defaultLanguage will increase the need of
it. I have also added XWikiDocument#getTranslatedDocument() with Locale in
place of language. All the changes are almost backward compatible, which is
nice (there is some subtleties with default, "" and null that is now more
equivalent, but should not have consequences).
The is however one change that is not backward compatible, which is the
change of the document reference. Therefore,
XWikiDocument#getDocumentReference does not return the same reference than
it does before, because this reference now contains the Locale. This cause
breakage in several places. I see some option to fix this:
A. Fix all places broken. This may be too long for me, and not trivial.
B. Introduce a new XWikiDocument#getDocumentReferenceWithLocale() and
have XWikiDocument#getDocumentRefence() returns without Locale. Very easy,
but not nice.
C. Introduce a new XWikiDocument#getDocumentReferenceWithoutLocale() and
change all existing calls to XWikiDocument#getDocumentRefence() in platform
to use this one. Nicer, but this is not fully backward compatible.
Since I am on it right now, I would appreciate your opinion quickly.
WDYT ?
I am undecided between B and C
Well after more thinking, there is no simple rules and B is not fully safe
in some places. So I am more in favor of C, since it is the cleanest on the
long term. I am also thinking to implement a cache of the reference without
local for better efficiency.
Please give me your comment asap, especially if you strongly disagree on C
since I will start coding this.
--
Denis Gervalle
SOFTEC sa - CEO
eGuilde sarl - CTO
--
Denis Gervalle
SOFTEC sa - CEO
eGuilde sarl - CTO
1:49 p.m.
New subject: [xwiki-devs] [VOTE] Adding the Locale into the document reference of XWikiDocuments
On Dec 9, 2011, at 1:41 PM, Denis Gervalle wrote:
On Fri, Dec 9, 2011 at 10:17, Denis Gervalle
<dgl(a)softec.lu> wrote:
> Hi Devs,
>
> I am looking at now using the new Locale added in DocumentReference into
> the implementation of XWikiDocument.
> I have already deprecated language related stuff in XWikiDocument, and I
> have introduce a XWikiDocument#getLocale and an XWikiDocument#isTranslation
> helper since the deprecation of defaultLanguage will increase the need of
> it. I have also added XWikiDocument#getTranslatedDocument() with Locale in
> place of language. All the changes are almost backward compatible, which is
> nice (there is some subtleties with default, "" and null that is now more
> equivalent, but should not have consequences).
Make sure you move the deprecated sutff in -legacy if you can.
> The is however one change that is not backward
compatible, which is the
> change of the document reference. Therefore,
> XWikiDocument#getDocumentReference does not return the same reference than
> it does before, because this reference now contains the Locale. This cause
> breakage in several places.
Why? It should be the same since the new ref only adds the optional locale.
> I see some option to fix this:
>
> A. Fix all places broken. This may be too long for me, and not trivial.
+1 if I understand correctly.
> B. Introduce a new
XWikiDocument#getDocumentReferenceWithLocale() and
> have XWikiDocument#getDocumentRefence() returns without Locale. Very easy,
> but not nice.
Not nice -0
> C. Introduce a new
XWikiDocument#getDocumentReferenceWithoutLocale() and
> change all existing calls to XWikiDocument#getDocumentRefence() in platform
> to use this one. Nicer, but this is not fully backward compatible.
-0 till I understand the problem above.
I don't understand the issue.
Thanks
-Vincent
Since I am on
it right now, I would appreciate your opinion quickly.
WDYT ?
I am undecided between B and C
Well after more thinking, there is no simple rules and B is not fully safe
in some places. So I am more in favor of C, since it is the cleanest on the
long term. I am also thinking to implement a cache of the reference without
local for better efficiency.
Please give me your comment asap, especially if you strongly disagree on C
since I will start coding this.
--
Denis Gervalle
SOFTEC sa - CEO
eGuilde sarl - CTO
--
Denis Gervalle
SOFTEC sa - CEO
eGuilde sarl - CTO
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
1:59 p.m.
New subject: [xwiki-devs] [VOTE] Adding the Locale into the document reference of XWikiDocuments
On Fri, Dec 9, 2011 at 13:49, Vincent Massol <vincent(a)massol.net> wrote:
On Dec 9, 2011, at 1:41 PM, Denis Gervalle wrote:
The issue is simple, the DocumentReference will now contains one more
information, the Locale, and this could be unexpected in some place where a
more generic reference (without locale) is required. Therefore, in some
place, the comparison of a DocumentReference, with the one coming from a
document may return false negative, because the comparison is done with a
reference without Locale.
I should have precise that I am applying C with a some review of each
usage, to not replace massively where this seems not necessary, and keep as
much as possible getDocumentReference in preference to
getDocumentReferenceWithoutLocale.
On Fri, Dec 9, 2011 at 10:17, Denis Gervalle
<dgl(a)softec.lu> wrote:
> Hi Devs,
>
> I am looking at now using the new Locale added in DocumentReference into
> the implementation of XWikiDocument.
> I have already deprecated language related stuff in XWikiDocument, and I
> have introduce a XWikiDocument#getLocale and an
XWikiDocument#isTranslation
> helper since the deprecation of
defaultLanguage will increase the need
of
> it. I have also added
XWikiDocument#getTranslatedDocument() with Locale
in
> place of language. All the changes are almost
backward compatible,
which is
> nice (there is some subtleties with default,
"" and null that is now
more
> equivalent, but should not have
consequences).
Make sure you move the deprecated sutff in -legacy if you can.
> The is however one change that is not
backward compatible, which is the
> change of the document reference. Therefore,
> XWikiDocument#getDocumentReference does not return the same reference
than
> it does before, because this reference now
contains the Locale. This
cause
> breakage in several places.
Why? It should be the same since the new ref only adds the optional locale.
> I see some option to fix this:
>
> A. Fix all places broken. This may be too long for me, and not trivial.
+1 if I understand correctly.
> B. Introduce a new
XWikiDocument#getDocumentReferenceWithLocale() and
> have XWikiDocument#getDocumentRefence() returns without Locale. Very
easy,
> but not nice.
Not nice -0
> C. Introduce a new
XWikiDocument#getDocumentReferenceWithoutLocale() and
> change all existing calls to XWikiDocument#getDocumentRefence() in
platform
> to use this one. Nicer, but this is not fully
backward compatible.
-0 till I understand the problem above.
I don't understand the issue.
Thanks
-Vincent
safe
--
Denis Gervalle
SOFTEC sa - CEO
eGuilde sarl - CTO
Since I
am on it right now, I would appreciate your opinion quickly.
WDYT ?
I am undecided between B and C
Well after more thinking, there is no simple rules and B is not fully in some places. So I am more in favor of C, since
it is the cleanest on
the
long term. I am also thinking to implement a
cache of the reference
without
local for better efficiency.
Please give me your comment asap, especially if you strongly disagree on
C
since I will start coding this.
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
--
Denis Gervalle
SOFTEC sa - CEO
eGuilde sarl - CTO
--
Denis Gervalle
SOFTEC sa - CEO
eGuilde sarl - CTO
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
3:03 p.m.
New subject: [xwiki-devs] [VOTE] Adding the Locale into the document reference of XWikiDocuments
On Dec 9, 2011, at 1:59 PM, Denis Gervalle wrote:
On Fri, Dec 9, 2011 at 13:49, Vincent Massol
<vincent(a)massol.net> wrote:
hmmm I'm ok with C but I don't see why you need to introduce a new API.
Why not offer an API to extract the document reference without the locale part?
I only recall we talked about a serializer for this (or a resolver, don't remember).
You may even have introduced an extractXXX method in DocumentReference to return a
refernce without the locale, I don't remember.
So why not update the code to use that instead of introducing a new "temporary"
API?
Thanks
-Vincent
On Dec 9, 2011, at 1:41 PM, Denis Gervalle wrote:
The issue is simple, the DocumentReference will now contains one more
information, the Locale, and this could be unexpected in some place where a
more generic reference (without locale) is required. Therefore, in some
place, the comparison of a DocumentReference, with the one coming from a
document may return false negative, because the comparison is done with a
reference without Locale.
I should have precise that I am applying C with a some review of each
usage, to not replace massively where this seems not necessary, and keep as
much as possible getDocumentReference in preference to
getDocumentReferenceWithoutLocale. On Fri, Dec 9, 2011 at 10:17, Denis Gervalle
<dgl(a)softec.lu> wrote:
> Hi Devs,
>
> I am looking at now using the new Locale added in DocumentReference into
> the implementation of XWikiDocument.
> I have already deprecated language related stuff in XWikiDocument, and I
> have introduce a XWikiDocument#getLocale and an
XWikiDocument#isTranslation
> helper since the deprecation of
defaultLanguage will increase the need
of
> it. I have also added
XWikiDocument#getTranslatedDocument() with Locale
in
> place of language. All the changes are almost
backward compatible,
which is
> nice (there is some subtleties with default,
"" and null that is now
more
> equivalent, but should not have
consequences).
Make sure you move the deprecated sutff in -legacy if you can.
> The is however one change that is not
backward compatible, which is the
> change of the document reference. Therefore,
> XWikiDocument#getDocumentReference does not return the same reference
than
> it does before, because this reference now
contains the Locale. This
cause
> breakage in several places.
Why? It should be the same since the new ref only adds the optional locale.
> I see some option to fix this:
>
> A. Fix all places broken. This may be too long for me, and not trivial.
+1 if I understand correctly.
> B. Introduce a new
XWikiDocument#getDocumentReferenceWithLocale() and
> have XWikiDocument#getDocumentRefence() returns without Locale. Very
easy,
> but not nice.
Not nice -0
> C. Introduce a new
XWikiDocument#getDocumentReferenceWithoutLocale() and
> change all existing calls to XWikiDocument#getDocumentRefence() in
platform
> to use this one. Nicer, but this is not fully
backward compatible.
-0 till I understand the problem above.
I don't understand the issue.
> Thanks
> -Vincent
>
>>> Since I am on it right now, I would appreciate your opinion quickly.
>>> WDYT ?
>>>
>>> I am undecided between B and C
>>>
>>
>> Well after more thinking, there is no simple rules and B is not fully
> safe
>> in some places. So I am more in favor of C, since it is the cleanest on
> the
>> long term. I am also thinking to implement a cache of the reference
> without
>> local for better efficiency.
>> Please give me your comment asap, especially if you strongly disagree on
> C
>> since I will start coding this.
>>
3:38 p.m.
New subject: [xwiki-devs] [VOTE] Adding the Locale into the document reference of XWikiDocuments
On Fri, Dec 9, 2011 at 15:03, Vincent Massol <vincent(a)massol.net> wrote:
On Dec 9, 2011, at 1:59 PM, Denis Gervalle wrote:
The issue is simple, the DocumentReference will now contains one more
information, the Locale, and this could be unexpected in some place where a
we have a constructor for that: new DocumentReference(docref, locale),
where locale set to null, remove the locale of the cloned reference.
On Fri, Dec 9, 2011 at 13:49, Vincent Massol
<vincent(a)massol.net> wrote:
>
> On Dec 9, 2011, at 1:41 PM, Denis Gervalle wrote:
>
>> On Fri, Dec 9, 2011 at 10:17, Denis Gervalle <dgl(a)softec.lu> wrote:
>>
>>> Hi Devs,
>>>
>>> I am looking at now using the new Locale added in DocumentReference
into
>>> the implementation of XWikiDocument.
>>> I have already deprecated language related stuff in XWikiDocument,
and
I
>>> have introduce a
XWikiDocument#getLocale and an
> XWikiDocument#isTranslation
>>> helper since the deprecation of defaultLanguage will increase the need
> of
>>> it. I have also added XWikiDocument#getTranslatedDocument() with
Locale
> in
>>> place of language. All the changes are almost backward compatible,
> which is
>>> nice (there is some subtleties with default, "" and null that is
now
> more
>>> equivalent, but should not have consequences).
>
> Make sure you move the deprecated sutff in -legacy if you can.
>
>>> The is however one change that is not backward compatible, which is
the
>>> change of the document reference.
Therefore,
>>> XWikiDocument#getDocumentReference does not return the same reference
> than
>>> it does before, because this reference now contains the Locale. This
> cause
>>> breakage in several places.
>
> Why? It should be the same since the new ref only adds the optional
locale.
>
>>> I see some option to fix this:
>>>
>>> A. Fix all places broken. This may be too long for me, and not
trivial.
>
> +1 if I understand correctly.
>
>>> B. Introduce a new XWikiDocument#getDocumentReferenceWithLocale() and
>>> have XWikiDocument#getDocumentRefence() returns without Locale. Very
> easy,
>>> but not nice.
>
> Not nice -0
>
>>> C. Introduce a new XWikiDocument#getDocumentReferenceWithoutLocale()
and
> change all existing calls to
XWikiDocument#getDocumentRefence() in
platform
> to use this one. Nicer, but this is not fully
backward compatible.
-0 till I understand the problem above.
I don't understand the issue.
more generic reference (without locale) is
required. Therefore, in some
place, the comparison of a DocumentReference, with the one coming from a
document may return false negative, because the comparison is done with a
reference without Locale.
I should have precise that I am applying C with a some review of each
usage, to not replace massively where this seems not necessary, and keep
as
much as possible getDocumentReference in
preference to
getDocumentReferenceWithoutLocale.
hmmm I'm ok with C but I don't see why you need to introduce a new API.
Why not offer an API to extract the document reference without the locale
part?
I only recall we talked about a serializer for this (or a resolver, don't
remember).
You may even have introduced an extractXXX method in DocumentReference to
return a refernce without the locale, I don't remember.
So why not update the code to use that instead of
introducing a new
"temporary" API?
But doing it often, with immutable refs, will be not really nice since it
create many similar reference which are all immutable, so why not keeping a
single instance in the document ? (that reduce memory footprint and speed
up comparisons)
BUT the problem is even larger than I had expected, and I am finally not
sure that even C will do.
When a reference to a document is retained in a XObject for exemple it
should not contains the locale. If we need either to clean the reference at
the setter or provide the document itself, so the setter can retrieve the
appropriate reference from it (which introduce deeper API change).
Compare to the target model (or what I understand of it), we currently have
an "original" document with potentially some "translated" documents.
The
original document may change its language at any time, but it will stay the
reference document for all links to it. And this reference should therefore
be without language. At the storage level, we have precisely that
representation in the docids. The docid of the "original" document does not
"contains" the language, so we can retrieve it quickly with hibernate
without knowing its language.
So finally, to not cause major breakage in the current code, my best bet
would be to not store any locale in the "original" document reference, and
to only add locale to the translated document references. I agree that this
is not perfect (not what I dream of), but this is the best representation
of what we currently have, since an "original" document may change its
language at anytime while staying the "original" document.
WDYT ?
Thanks
-Vincent
--
Denis Gervalle
SOFTEC sa - CEO
eGuilde sarl - CTO
> Thanks
> -Vincent
>
>>> Since I am on it right now, I would appreciate your opinion quickly.
>>> WDYT ?
>>>
>>> I am undecided between B and C
>>>
>>
>> Well after more thinking, there is no simple rules and B is not fully
> safe
>> in some places. So I am more in favor of C, since it is the cleanest on
> the
>> long term. I am also thinking to implement a cache of the reference
> without
>> local for better efficiency.
>> Please give me your comment asap, especially if you strongly disagree
on
> C
>> since I will start coding this.
>>
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
16 Dec
16 Dec
9:18 a.m.
New subject: [xwiki-devs] [VOTE] Adding the Locale into the document reference of XWikiDocuments
On Fri, Dec 9, 2011 at 10:17 AM, Denis Gervalle <dgl(a)softec.lu> wrote:
Hi Devs,
I am looking at now using the new Locale added in DocumentReference into
the implementation of XWikiDocument.
I have already deprecated language related stuff in XWikiDocument, and I
have introduce a XWikiDocument#getLocale and an XWikiDocument#isTranslation
helper since the deprecation of defaultLanguage will increase the need of
it. I have also added XWikiDocument#getTranslatedDocument() with Locale in
place of language. All the changes are almost backward compatible, which is
nice (there is some subtleties with default, "" and null that is now more
equivalent, but should not have consequences).
The is however one change that is not backward compatible, which is the
change of the document reference. Therefore,
XWikiDocument#getDocumentReference does not return the same reference than
it does before, because this reference now contains the Locale. This cause
breakage in several places. I see some option to fix this:
A. Fix all places broken. This may be too long for me, and not trivial.
B. Introduce a new XWikiDocument#getDocumentReferenceWithLocale() and
have XWikiDocument#getDocumentRefence() returns without Locale. Very easy,
but not nice.
C. Introduce a new XWikiDocument#getDocumentReferenceWithoutLocale() and
change all existing calls to XWikiDocument#getDocumentRefence() in platform
to use this one. Nicer, but this is not fully backward compatible.
Since I am on it right now, I would appreciate your opinion quickly.
WDYT ?
I am undecided between B and C
+1 for C
--
Denis Gervalle
SOFTEC sa - CEO
eGuilde sarl - CTO
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
--
Thomas Mortagne
4658
days inactive
4665
days old
6 comments
3 participants
participants (3)
-
Denis Gervalle -
Thomas Mortagne -
Vincent Massol