Fixes UserEmailAddressExportForm
authorMatthias Schmidt <gravatronics@live.com>
Mon, 7 Jan 2013 21:11:47 +0000 (22:11 +0100)
committerMatthias Schmidt <gravatronics@live.com>
Mon, 7 Jan 2013 21:11:47 +0000 (22:11 +0100)
wcfsetup/install/files/lib/acp/form/UserEmailAddressExportForm.class.php
wcfsetup/install/files/lib/acp/form/UserMailForm.class.php

index e43e9c66674c9dc78e4d6a0bab4ae1bdfdba3e30..c6fd36ce4c499e393efd6a1e3f93a08cf8a656f1 100755 (executable)
@@ -63,7 +63,7 @@ class UserEmailAddressExportForm extends AbstractForm {
         * clipboard item type id
         * @var integer
         */
-       protected $typeID = null;
+       protected $objectTypeID = null;
        
        /**
         * @see wcf\page\IPage::readParameters()
@@ -71,19 +71,21 @@ class UserEmailAddressExportForm extends AbstractForm {
        public function readParameters() {
                parent::readParameters();
                
-               // get type id
-               $this->typeID = ClipboardHandler::getInstance()->getObjectTypeID('com.woltlab.wcf.user');
-               if ($this->typeID === null) {
-                       throw new SystemException("clipboard item type 'com.woltlab.wcf.user' is unknown.");
+               // get object type id
+               $this->objectTypeID = ClipboardHandler::getInstance()->getObjectTypeID('com.woltlab.wcf.user');
+               if ($this->objectTypeID === null) {
+                       throw new SystemException("Unknown clipboard item type 'com.woltlab.wcf.user'");
                }
                
                // get user ids
-               $users = ClipboardHandler::getInstance()->getMarkedItems($this->typeID);
-               if (!isset($users['com.woltlab.wcf.user']) || empty($users['com.woltlab.wcf.user'])) throw new IllegalLinkException();
+               $users = ClipboardHandler::getInstance()->getMarkedItems($this->objectTypeID);
+               if (empty($users)) {
+                       throw new IllegalLinkException();
+               }
                
                // load users
-               $this->userIDs = array_keys($users['com.woltlab.wcf.user']);
-               $this->users = $users['com.woltlab.wcf.user'];
+               $this->userIDs = array_keys($users);
+               $this->users = $users;
        }
        
        /**
@@ -148,7 +150,7 @@ class UserEmailAddressExportForm extends AbstractForm {
                $this->saved();
                
                // remove items
-               ClipboardHandler::getInstance()->removeItems($this->typeID);
+               ClipboardHandler::getInstance()->removeItems($this->objectTypeID);
                
                exit;
        }
index f7c44c5f7cffc6b0d3725abb7dc5da0b29c5a16c..f9d60e67cfda48b5d1dd836487392d0ddf76ec39 100755 (executable)
@@ -174,7 +174,7 @@ class UserMailForm extends AbstractForm {
                                $this->userIDs = array_keys($users);
                        }
                        
-                       if (MAIL_USE_FORMATTED_ADDRESS) {
+                       if (MAIL_USE_FORMATTED_ADDRESS) {
                                $this->from = MAIL_FROM_NAME.' <'.MAIL_FROM_ADDRESS.'>';
                        }
                        else {