ElkArte Community

Did it indicate which line the error is on?

ManageAttachments.controller.php

Line: 218

					$modSettings['attachmentUploadDir'] = !empty($modSettings['attachmentUploadDir']) ?: array();

Hummm .... try adding the following line just above that line ( in ManaageAttachments.controller.php )

Code: [Select]

					$modSettings['attachmentUploadDir'] = !empty($modSettings['attachmentUploadDir']) ?: array();

and lets see what happens

Warning: in_array() expects parameter 2 to be array, bool given

					if (empty($modSettings['attachmentUploadDir']))
					{
						$modSettings['attachmentUploadDir'] = array();
					}
					else
					{
						$test = @Util::unserialize($modSettings['attachmentUploadDir']);
						if ($test === false)
						{
							$modSettings['attachmentUploadDir'] = array($modSettings['attachmentUploadDir']);
						}
						else
						{
							$modSettings['attachmentUploadDir'] = $test;
						}
					}

if (isset($this->_req->post->attachmentUploadDir))

			if (isset($this->_req->post->attachmentUploadDir) ||
				!empty($this->_req->post->use_subdirectories_for_attachments))

Maybe...
Line 190:

Code: [Select]

if (isset($this->_req->post->attachmentUploadDir))

to:

Code: [Select]

			if (isset($this->_req->post->attachmentUploadDir) ||
				!empty($this->_req->post->use_subdirectories_for_attachments))

Humm .... this is why @emanuele is updating this code for 2.0

Just for a test, try using

Code: [Select]

					if (empty($modSettings['attachmentUploadDir']))
					{
						$modSettings['attachmentUploadDir'] = array();
					}
					else
					{
						$test = @Util::unserialize($modSettings['attachmentUploadDir']);
						if ($test === false)
						{
							$modSettings['attachmentUploadDir'] = array($modSettings['attachmentUploadDir']);
						}
						else
						{
							$modSettings['attachmentUploadDir'] = $test;
						}
					}

instead of the other single line I gave you.

Yeah so the code is a bit broken ....  but I think you know that 

FYI what I personally do is create directories for each year, and I manually create them   I don't really see the need for monthly separation unless you were on a site that created an enormous number of attachments.  I break it down by year just to simplify my backup scripts.

In 2.0 the plan was to simply that code, it simply allows to many options which causes complications for everyone.

I think the most common limit to avoid is the usually 10k files per directory that is enforced for ftp in certain configurations (i.e. the ftp returns only the first 10k files). Though this limit is not always present, so it's a kind of case-by-case.
Apart from that, it could be wise to avoid to have more than 10k files in a directory just to remain sane.