SF.net SVN: phpwiki:[11063] trunk/lib
vargenau--- via phpwiki-checkins <[email protected]> Tue, 18 Jul 2023 14:37:59 +0000
| Newsgroups | gmane.comp.web.wiki.phpwiki.checkins |
|---|---|
| Message-ID | <[email protected]> |
Revision: 11063
http://sourceforge.net/p/phpwiki/code/11063
Author: vargenau
Date: 2023-07-18 14:37:58 +0000 (Tue, 18 Jul 2023)
Log Message:
-----------
PHP7: less strict typing
Modified Paths:
--------------
trunk/lib/PageList.php
trunk/lib/Template.php
trunk/lib/editpage.php
Modified: trunk/lib/PageList.php
===================================================================
--- trunk/lib/PageList.php 2023-07-18 14:36:37 UTC (rev 11062)
+++ trunk/lib/PageList.php 2023-07-18 14:37:58 UTC (rev 11063)
@@ -85,7 +85,7 @@
}
// old-style heading
- public function heading(): HTML
+ public function heading()
{
global $request;
// allow sorting?
@@ -110,7 +110,7 @@
// new grid-style sortable heading
// TODO: via activeui.js ? (fast dhtml sorting)
- public function button_heading($pagelist, $colNum): HTML
+ public function button_heading($pagelist, $colNum)
{
global $WikiTheme, $request;
// allow sorting?
@@ -209,7 +209,7 @@
* @param WikiDB_PageRevision $revision_handle
* @return mixed
*/
- public function _getValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getValue($page_handle, $revision_handle)
{
if ($this->_need_rev) {
if (!$revision_handle) {
@@ -227,7 +227,7 @@
* @param WikiDB_PageRevision $revision_handle
* @return int|string
*/
- public function _getSortableValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getSortableValue($page_handle, $revision_handle)
{
$val = $this->_getValue($page_handle, $revision_handle);
if ($this->_field == 'hits') {
@@ -256,7 +256,7 @@
class _PageList_Column_size extends _PageList_Column
{
- public function format($pagelist, $page_handle, $revision_handle): HTML
+ public function format($pagelist, $page_handle, $revision_handle)
{
return HTML::td(
$this->_tdattr,
@@ -264,7 +264,7 @@
);
}
- public function _getValuePageList($pagelist, $page_handle, $revision_handle): FormattedText
+ public function _getValuePageList($pagelist, $page_handle, $revision_handle)
{
if (!$revision_handle or (!$revision_handle->_data['%content']
or $revision_handle->_data['%content'] === true)
@@ -280,7 +280,7 @@
return $size;
}
- public function _getSortableValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getSortableValue($page_handle, $revision_handle)
{
if (!$revision_handle) {
$revision_handle = $page_handle->getCurrentRevision();
@@ -305,7 +305,7 @@
$this->_textIfFalse = new RawXml('—'); //mdash
}
- public function _getValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getValue($page_handle, $revision_handle)
{
//FIXME: check if $this is available in the parent (->need_rev)
$val = parent::_getValue($page_handle, $revision_handle);
@@ -327,7 +327,7 @@
parent::__construct($field, $heading, 'center');
}
- public function _getValuePageList($pagelist, $page_handle, $revision_handle): HTML
+ public function _getValuePageList($pagelist, $page_handle, $revision_handle)
{
$pagename = $page_handle->getName();
$selected = !empty($pagelist->_selected[$pagename]);
@@ -346,7 +346,7 @@
}
}
- public function format($pagelist, $page_handle, $revision_handle): HTML
+ public function format($pagelist, $page_handle, $revision_handle)
{
return HTML::td(
$this->_tdattr,
@@ -355,7 +355,7 @@
}
// don't sort this javascript button
- public function button_heading($pagelist, $colNum): HTML
+ public function button_heading($pagelist, $colNum)
{
$s = HTML($this->_heading);
return HTML::th(array('class' => 'gridbutton'), $s);
@@ -371,13 +371,13 @@
$this->WikiTheme = &$WikiTheme;
}
- public function _getValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getValue($page_handle, $revision_handle)
{
$time = parent::_getValue($page_handle, $revision_handle);
return $this->WikiTheme->formatDateTime($time);
}
- public function _getSortableValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getSortableValue($page_handle, $revision_handle)
{
return parent::_getValue($page_handle, $revision_handle);
}
@@ -385,7 +385,7 @@
class _PageList_Column_version extends _PageList_Column
{
- public function _getValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getValue($page_handle, $revision_handle)
{
if (!$revision_handle) {
$revision_handle = $page_handle->getCurrentRevision();
@@ -425,7 +425,7 @@
}
}
- public function _getValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getValue($page_handle, $revision_handle)
{
if (!$revision_handle or (!$revision_handle->_data['%content']
or $revision_handle->_data['%content'] === true)
@@ -510,7 +510,7 @@
);
}
- public function _getSortableValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getSortableValue($page_handle, $revision_handle)
{
if (is_object($page_handle) and !empty($page_handle->score)) {
return $page_handle->score;
@@ -535,7 +535,7 @@
$this->dbi =& $request->getDbh();
}
- public function _getValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getValue($page_handle, $revision_handle)
{
$author = parent::_getValue($page_handle, $revision_handle);
if ($this->dbi->isWikiPage($author)) {
@@ -545,7 +545,7 @@
}
}
- public function _getSortableValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getSortableValue($page_handle, $revision_handle)
{
return parent::_getValue($page_handle, $revision_handle);
}
@@ -553,7 +553,7 @@
class _PageList_Column_owner extends _PageList_Column_author
{
- public function _getValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getValue($page_handle, $revision_handle)
{
$author = $page_handle->getOwner();
if ($this->dbi->isWikiPage($author)) {
@@ -563,7 +563,7 @@
}
}
- public function _getSortableValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getSortableValue($page_handle, $revision_handle)
{
return parent::_getValue($page_handle, $revision_handle);
}
@@ -571,7 +571,7 @@
class _PageList_Column_creator extends _PageList_Column_author
{
- public function _getValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getValue($page_handle, $revision_handle)
{
$author = $page_handle->getCreator();
if ($this->dbi->isWikiPage($author)) {
@@ -581,7 +581,7 @@
}
}
- public function _getSortableValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getSortableValue($page_handle, $revision_handle)
{
return parent::_getValue($page_handle, $revision_handle);
}
@@ -598,7 +598,7 @@
$this->dbi = &$request->getDbh();
}
- public function _getValue($page_handle, $revision_handle): XmlContent
+ public function _getValue($page_handle, $revision_handle)
{
if ($this->dbi->isWikiPage($page_handle->getName())) {
return WikiLink($page_handle);
@@ -623,7 +623,7 @@
class _PageList_Column_perm extends _PageList_Column
{
- public function _getValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getValue($page_handle, $revision_handle)
{
$perm_array = pagePermissions($page_handle->_pagename);
return pagePermissionsSimpleFormat(
@@ -636,7 +636,7 @@
class _PageList_Column_acl extends _PageList_Column
{
- public function _getValue(WikiDB_Page $page_handle, WikiDB_PageRevision $revision_handle)
+ public function _getValue($page_handle, $revision_handle)
{
$perm_tree = pagePermissions($page_handle->_pagename);
@@ -1738,7 +1738,7 @@
}
// make a table given the caption
- public function _generateTable($caption = ''): HTML
+ public function _generateTable($caption = '')
{
if (count($this->_sortby) > 0) {
$this->sortPages();
@@ -2010,7 +2010,7 @@
// comma=1
// Condense list without a href links: "Page1, Page2, ..."
// FIXME: only unique list entries, esp. with nopage
- private function generateCommaListAsString(): XmlContent
+ private function generateCommaListAsString()
{
if (defined($this->_options['commasep'])) {
$separator = $this->_options['commasep'];
@@ -2031,7 +2031,7 @@
// Future: 1 = reserved for plain string (see above)
// 2 and more => HTML link specialization?
// FIXME: only unique list entries, esp. with nopage
- private function generateCommaList(): XmlContent
+ private function generateCommaList()
{
if (defined($this->_options['commasep'])) {
$separator = HTML::raw($this->_options['commasep']);
@@ -2049,7 +2049,7 @@
return $html;
}
- public function _emptyList($caption): XmlContent
+ public function _emptyList($caption)
{
$html = HTML();
if ($caption) {
Modified: trunk/lib/Template.php
===================================================================
--- trunk/lib/Template.php 2023-07-18 14:36:37 UTC (rev 11062)
+++ trunk/lib/Template.php 2023-07-18 14:37:58 UTC (rev 11063)
@@ -37,7 +37,7 @@
* @param WikiRequest $request
* @param array $args
*/
- public function __construct(string $name, &$request, array $args = array())
+ public function __construct(string $name, &$request, $args = array())
{
global $WikiTheme;
Modified: trunk/lib/editpage.php
===================================================================
--- trunk/lib/editpage.php 2023-07-18 14:36:37 UTC (rev 11062)
+++ trunk/lib/editpage.php 2023-07-18 14:37:58 UTC (rev 11063)
@@ -57,7 +57,7 @@
/**
* @param WikiRequest $request
*/
- public function __construct(WikiRequest &$request)
+ public function __construct(&$request)
{
$this->request = &$request;
@@ -123,7 +123,7 @@
}
}
- public function editPage(): ?bool
+ public function editPage()
{
$saveFailed = false;
$tokens = &$this->tokens;
@@ -540,7 +540,7 @@
/** Header of the Anti Spam message
*/
- private function getSpamMessage(): XmlContent
+ private function getSpamMessage()
{
return
HTML(
@@ -554,7 +554,7 @@
);
}
- protected function getPreview(): TransformedText
+ protected function getPreview()
{
require_once 'lib/PageType.php';
$this->_content = $this->getContent();
@@ -561,7 +561,7 @@
return new TransformedText($this->page, $this->_content, $this->meta);
}
- protected function getConvertedPreview(): TransformedText
+ protected function getConvertedPreview()
{
require_once 'lib/PageType.php';
$this->_content = $this->getContent();
@@ -568,7 +568,7 @@
return new TransformedText($this->page, $this->_content, $this->meta);
}
- private function getDiff(): XmlContent
+ private function getDiff()
{
require_once 'lib/diff.php';
$html = HTML();
@@ -611,7 +611,7 @@
}
}
- protected function getLockedMessage(): XmlContent
+ protected function getLockedMessage()
{
return
HTML(
@@ -627,7 +627,7 @@
return $this->page->get('moderation');
}
- private function getModeratedMessage(): XmlContent
+ private function getModeratedMessage()
{
return
HTML(
@@ -640,7 +640,7 @@
);
}
- protected function getConflictMessage($unresolved = false): XmlContent
+ protected function getConflictMessage($unresolved = false)
{
/*
xgettext only knows about c/c++ line-continuation strings
@@ -1255,7 +1255,7 @@
class LoadFileConflictPageEditor extends PageEditor
{
- public function editPage($saveFailed = true): ?bool
+ public function editPage($saveFailed = true)
{
$tokens = &$this->tokens;
@@ -1371,7 +1371,7 @@
return true;
}
- protected function getConflictMessage($unresolved = false): XmlContent
+ protected function getConflictMessage($unresolved = false)
{
return HTML(HTML::p(
fmt(
This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.