Last modified: 2012-02-23 15:57:34 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 T36499, the corresponding Phabricator task for complete and up-to-date bug report information.

Bug 34499 - SemanticMediaWiki: r111407 breaks creating new pages.


Summary:	SemanticMediaWiki: r111407 breaks creating new pages.

Status:	RESOLVED FIXED

Product:	MediaWiki extensions
Classification:	Unclassified
Component:	Semantic MediaWiki (Other open bugs)
Version:	unspecified
Hardware:	All All

Importance:	Unprioritized normal (vote)
Target Milestone:	---
Assigned To:	Jeroen De Dauw

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:
	Show dependency tree / graph

Reported:	2012-02-18 19:41 UTC by Van de Bugger
Modified:	2012-02-23 15:57 UTC (History)
CC List:	2 users (show)

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Van de Bugger 2012-02-18 19:41:11 UTC

r111407 by jeroendedauw "some work on special property handling" breaks creating new pages. Saving a new page causes 

> Fatal error: Call to a member function getUser() on a non-object in
> .../Extensions/SemanticMediaWiki-trunk/includes/SMW_ParseData.php on line 446

Comment 1 Jeroen De Dauw 2012-02-18 19:49:18 UTC

I can't reproduce this. Can you link to where this is happening? If not, what version of MW are you using?

Comment 2 Van de Bugger 2012-02-22 21:30:31 UTC

MW 1.18.0.

It is happening on my site running on localhost. No access from outside, sorry.

BTW, why did you change type of `_NEWP' from `_boo' to `_dat'?? This property is completely broken now, because other code sets it to boolean value:

> case '_NEWP' :
>     $value = new SMWDIBoolean( $title->isNewPage() );

Comment 3 Jeroen De Dauw 2012-02-23 12:36:24 UTC

> BTW, why did you change type of `_NEWP' from `_boo' to `_dat'??

Debugging, but I should have changed it back. *sigh* this is why we need automated tests :)

Can you confirm r112197 fixes this bug?

Comment 4 Van de Bugger 2012-02-23 15:54:59 UTC

Ok, now everything looks ok to me. Both _NEWP and _LEDT work as expected.

Comment 5 Jeroen De Dauw 2012-02-23 15:57:34 UTC

Great - thanks again for adding this in :)

Wikimedia Bugzilla is closed!

Search

Personal tools

Navigation

Links