]> git.mxchange.org Git - quix0rs-gnu-social.git/commitdiff
Merge branch 'nightly' of gitorious.org:social/mainline into nightly
authorRoland Haeder <roland@mxchange.org>
Fri, 6 Mar 2015 23:34:34 +0000 (00:34 +0100)
committerRoland Haeder <roland@mxchange.org>
Fri, 6 Mar 2015 23:34:34 +0000 (00:34 +0100)
Better use a clear type-hint and array() as default value then you can avoid the
additional is_null() call.

Conflicts:
lib/noticeform.php

Signed-off-by: Roland Haeder <roland@mxchange.org>
1  2 
lib/action.php
lib/default.php
lib/noticeform.php

diff --cc lib/action.php
Simple merge
diff --cc lib/default.php
Simple merge
index 9fc612a775f7ae702178db2e3c134a08085d24a2,4b3801b6301be74aada650117c2b2d0cfb8781e8..a92b431dd6bd8edd7ab77e48ed3a121e5f65d6f6
@@@ -97,16 -97,18 +97,14 @@@ class NoticeForm extends For
       *                        'location_id' ID of location
       *                        'location_ns' Namespace of location
       */
 -    function __construct($action, $options=null)
 +    function __construct(Action $action, array $options = array())
      {
-         // XXX: ??? Is this to keep notice forms distinct?
-         // Do we have to worry about sub-second race conditions?
-         // XXX: Needs to be above the parent::__construct() call...?
+         parent::__construct($action);
  
+         // When creating a notice form we don't want to collide with
+         // possibly existing HTML elements, as naming conventions are similar.
          $this->id_suffix = rand();
  
-         parent::__construct($action);
 -        if (is_null($options)) {
 -            $options = array();
 -        }
--
          $this->actionName  = $action->trimmed('action');
  
          $prefill = array('content', 'inreplyto', 'lat',