Last modified: 2009-07-31 14:35:44 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 T19848, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 17848 - Property E-Mail
Property E-Mail
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Semantic MediaWiki (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: DaSch
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-03-07 20:13 UTC by DaSch
Modified: 2009-07-31 14:35 UTC (History)
1 user (show)

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


Attachments
Small Patch (719 bytes, patch)
2009-03-07 21:05 UTC, DaSch
Details
Patch for Type Email (1.03 KB, patch)
2009-03-07 21:49 UTC, DaSch
Details

Description DaSch 2009-03-07 20:13:03 UTC
I think mailto: should not be displayed because it's unnecessary to have a lot of space taking mailto in tables for example
Comment 1 DaSch 2009-03-07 21:05:51 UTC
Created attachment 5901 [details]
Small Patch

I made this small patch, but this only fixes the Problem inside factboxes but has no effect on inline-query output
Comment 2 DaSch 2009-03-07 21:31:01 UTC
Well the behaviour is really strange because every null-edit changes the output

Another thig is that the getShortWikiText and getShortHTMLText is a bit strange to me

if ( ($linked === NULL) || ($linked === false) || ($this->m_url == '') || ($this->m_caption == '') ) {
	return $this->m_caption;

Is this right that when m_caption is empty m_caption is printed?
Comment 3 DaSch 2009-03-07 21:49:08 UTC
Created attachment 5902 [details]
Patch for Type Email

So now it works. DB stores E-Mail without mailto: (save some resource) mailto: is given by type Mail

The output works the way that mailto: is added to E-Mail URL and the visible value is the one storred in Database
Comment 4 Markus Krötzsch 2009-03-27 15:43:57 UTC
(In reply to comment #2)
> Well the behaviour is really strange because every null-edit changes the output
> 
> Another thig is that the getShortWikiText and getShortHTMLText is a bit strange
> to me
> 
> if ( ($linked === NULL) || ($linked === false) || ($this->m_url == '') ||
> ($this->m_caption == '') ) {
>         return $this->m_caption;
> 
> Is this right that when m_caption is empty m_caption is printed?
> 

Yes, this is how it is possible to hide annotations by giving only a space as alternative text.
Comment 5 Markus Krötzsch 2009-03-27 15:49:17 UTC
(In reply to comment #3)
> Created an attachment (id=5902) [details]
> Patch for Type Email
> 
> So now it works. DB stores E-Mail without mailto: (save some resource) mailto:
> is given by type Mail
> 
> The output works the way that mailto: is added to E-Mail URL and the visible
> value is the one storred in Database
> 

This does not work yet: it breaks the RDF export (where mailto: is needed, too). I need to consider this carefully; changing the DB contents is always a bit tricky (everybody needs to refresh the DB on upgrade!), and modifying the default representation may break other features unexpectedly. But I think that it is doable.
Comment 6 Markus Krötzsch 2009-07-31 14:35:44 UTC
I have now fixed this in SVN. "mailto:" is optional in all email inputs, and it is generally omitted in outputs.

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


Navigation
Links