Last modified: 2014-09-11 08:41:19 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T67847, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 65847 - not possible to switch to Julian calendar model
not possible to switch to Julian calendar model
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
WikidataRepo (Other open bugs)
master
All All
: High normal (vote)
: ---
Assigned To: Wikidata bugs
u=dev c=frontend p=5 s=2014-08-26
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-05-28 10:40 UTC by Lydia Pintscher
Modified: 2014-09-11 08:41 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Lydia Pintscher 2014-05-28 10:40:59 UTC
We broke switching calendar model it seems. One example is the date of birth in https://www.wikidata.org/wiki/Q1977210. Clicking "change to Julian" in the input expander does nothing.
Comment 1 Adrian Lang 2014-06-02 13:04:41 UTC
Seems to be fixed for me with https://github.com/wmde/ValueView/pull/51.
Comment 2 Lydia Pintscher 2014-07-14 14:10:15 UTC
The above patch is live and the issue is still there. It seems to be something else.
Comment 3 WikidataBugzilla 2014-07-14 14:52:56 UTC
Commit pushed to julian at https://github.com/wmde/DataValuesJavascript

https://github.com/wmde/DataValuesJavascript/commit/c4fb292c1e7289f18a33561f67999158ad11e04d
Fix incomplete Time.equals

Bug: 65847
Comment 4 Lydia Pintscher 2014-08-19 10:28:07 UTC
We just tried this and none of the above patches seem to fix it.
Comment 5 WikidataBugzilla 2014-08-27 12:17:43 UTC
Commit pushed to fix-calendar-switch at https://github.com/wmde/ValueView

https://github.com/wmde/ValueView/commit/eb2f8eff1fa5a824da0ec4b2ecb102a12b19371d
Fix calendar switch to Julian (Bug 65847).
Comment 6 Gerrit Notification Bot 2014-09-01 16:16:01 UTC
Change 157689 had a related patch set uploaded by Tobias Gritschacher:
Use new versions of DataValuesJavascript and ValueView

https://gerrit.wikimedia.org/r/157689
Comment 7 WikidataBugzilla 2014-09-02 16:05:19 UTC
Commit pushed to master at https://github.com/wmde/ValueView

https://github.com/wmde/ValueView/commit/4695925d785995841f090b95d2f992798c8bffa3
Merge pull request #109 from wmde/fix-calendar-switch

Fix calendar switch to Julian (Bug 65847).
Comment 8 JulesWinnfield-hu 2014-09-04 09:49:25 UTC
The UI is broken. It doesn't convert Julian dates to proleptic Gregorian values. Some time ago it worked fine.
Comment 9 Andre Klapper 2014-09-04 11:17:16 UTC
(In reply to JulesWinnfield-hu from comment #8)
> The UI is broken.

Steps to reproduce welcome.
Comment 10 JulesWinnfield-hu 2014-09-04 11:25:48 UTC
(In reply to Andre Klapper from comment #9)
Enter any date before 1500, try to set Julian calendar model, save, see the stored value. No conversion occures.
Comment 11 Henning 2014-09-04 12:17:07 UTC
(In reply to JulesWinnfield-hu from comment #10)
> (In reply to Andre Klapper from comment #9)
> Enter any date before 1500, try to set Julian calendar model, save, see the
> stored value. No conversion occures.

Thanks for your report. I examined the issue and opened a new bug report at bug #70395. If you have additional comments or questions, please add them there.
Closing this bug report again since the issue actually realted to this bug report is fixed (will be in the next deployment branch).
Comment 12 JulesWinnfield-hu 2014-09-04 12:44:57 UTC
(In reply to Henning from comment #11)
It's not only about display, though the display is also broken because no conversion occures either. It's about storing wrong date value.
This was good at a time. Why had to be changed? Also, dates before 1500 doesn't default to Julian like before.
Comment 13 tobias.gritschacher 2014-09-04 12:55:56 UTC
(In reply to JulesWinnfield-hu from comment #12)
> (In reply to Henning from comment #11)
> It's not only about display, though the display is also broken because no
> conversion occures either. It's about storing wrong date value.
> This was good at a time. Why had to be changed? Also, dates before 1500
> doesn't default to Julian like before.

Please note that there are several issues. The one described in this bug report has been fixed in https://github.com/wmde/ValueView/pull/109. For another one bug #70395 has been created. You say there is also an issue with storing the wrong data value? We need to investigate that and then open another bug. Or you open one in the meantime. Please do not reopen this bug again as this was only about an issue in the frontend. See the "frontend" tag in the whiteboard.
Comment 14 JulesWinnfield-hu 2014-09-05 17:36:23 UTC
Opened bug 70398.
Comment 15 Gerrit Notification Bot 2014-09-11 08:41:19 UTC
Change 157689 merged by jenkins-bot:
Use new versions of DataValuesJavascript and ValueView

https://gerrit.wikimedia.org/r/157689

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links