Allow Save As Copy Article In The Frontend (#35378)

Author: joomdonation

administrator/components/com_content/src/Model/ArticleModel.php

@@ -13,6 +13,8 @@
 
 use Joomla\CMS\Event\AbstractEvent;
 use Joomla\CMS\Factory;
+use Joomla\CMS\Filter\InputFilter;
+use Joomla\CMS\Filter\OutputFilter;
 use Joomla\CMS\Form\Form;
 use Joomla\CMS\Form\FormFactoryInterface;
 use Joomla\CMS\Helper\TagsHelper;
@@ -684,10 +686,9 @@ public function validate($form, $data, $group = null)
 	 */
 	public function save($data)
 	{
-		$input  = Factory::getApplication()->input;
-		$filter = \JFilterInput::getInstance();
-		$db     = $this->getDbo();
-		$user	= Factory::getUser();
+		$app    = Factory::getApplication();
+		$input  = $app->input;
+		$filter = InputFilter::getInstance();
 
 		if (isset($data['metadata']) && isset($data['metadata']['author']))
 		{
@@ -780,21 +781,39 @@ public function save($data)
 		// Alter the title for save as copy
 		if ($input->get('task') == 'save2copy')
 		{
-			$origTable = clone $this->getTable();
-			$origTable->load($input->getInt('id'));
+			$origTable = $this->getTable();
+
+			if ($app->isClient('site'))
+			{
+				$origTable->load($input->getInt('a_id'));
+
+				if ($origTable->title === $data['title'])
+				{
+					/**
+					 * If title of article is not changed, set alias to original article alias so that Joomla! will generate
+					 * new Title and Alias for the copied article
+					 */
+					$data['alias'] = $origTable->alias;
+				}
+				else
+				{
+					$data['alias'] = '';
+				}
+			}
+			else
+			{
+				$origTable->load($input->getInt('id'));
+			}
 
 			if ($data['title'] == $origTable->title)
 			{
 				list($title, $alias) = $this->generateNewTitle($data['catid'], $data['alias'], $data['title']);
 				$data['title'] = $title;
 				$data['alias'] = $alias;
 			}
-			else
+			elseif ($data['alias'] == $origTable->alias)
 			{
-				if ($data['alias'] == $origTable->alias)
-				{
-					$data['alias'] = '';
-				}
+				$data['alias'] = '';
 			}
 		}
 
@@ -803,16 +822,16 @@ public function save($data)
 		{
 			if ($data['alias'] == null)
 			{
-				if (Factory::getApplication()->get('unicodeslugs') == 1)
+				if ($app->get('unicodeslugs') == 1)
 				{
-					$data['alias'] = \JFilterOutput::stringUrlUnicodeSlug($data['title']);
+					$data['alias'] = OutputFilter::stringUrlUnicodeSlug($data['title']);
 				}
 				else
 				{
-					$data['alias'] = \JFilterOutput::stringURLSafe($data['title']);
+					$data['alias'] = OutputFilter::stringURLSafe($data['title']);
 				}
 
-				$table = Table::getInstance('Content', 'JTable');
+				$table = $this->getTable();
 
 				if ($table->load(array('alias' => $data['alias'], 'catid' => $data['catid'])))
 				{
@@ -824,7 +843,7 @@ public function save($data)
 
 				if (isset($msg))
 				{
-					Factory::getApplication()->enqueueMessage($msg, 'warning');
+					$app->enqueueMessage($msg, 'warning');
 				}
 			}
 		}

components/com_content/src/Controller/ArticleController.php

@@ -391,6 +391,10 @@ public function save($key = null, $urlVar = 'a_id')
 				$this->setRedirect(Route::_('index.php?Itemid=' . $menuitem . $lang, false));
 			}
 		}
+		elseif ($this->getTask() === 'save2copy')
+		{
+			// Redirect to the article page, use the redirect url set from parent controller
+		}
 		else
 		{
 			// If ok, redirect to the return page.

components/com_content/src/View/Form/HtmlView.php

@@ -86,6 +86,14 @@ class HtmlView extends BaseHtmlView
 	 */
 	protected $captchaEnabled = false;
 
+	/**
+	 * Should we show Save As Copy button?
+	 *
+	 * @var    boolean
+	 * @since  __DEPLOY_VERSION__
+	 */
+	protected $showSaveAsCopy = false;
+
 	/**
 	 * Execute and display a template script.
 	 *
@@ -95,8 +103,8 @@ class HtmlView extends BaseHtmlView
 	 */
 	public function display($tpl = null)
 	{
-		$user = Factory::getUser();
 		$app  = Factory::getApplication();
+		$user = $app->getIdentity();
 
 		// Get model data.
 		$this->state       = $this->get('State');
@@ -181,7 +189,15 @@ public function display($tpl = null)
 			}
 		}
 
+		// If the article is being edited and the current user has permission to create article
+		if ($this->item->id
+			&& ($user->authorise('core.create', 'com_content') || \count($user->getAuthorisedCategories('com_content', 'core.create'))))
+		{
+			$this->showSaveAsCopy = true;
+		}
+
 		$this->_prepareDocument();
+
 		parent::display($tpl);
 	}
 

components/com_content/tmpl/form/edit.php

@@ -161,6 +161,12 @@
 				<span class="icon-check" aria-hidden="true"></span>
 				<?php echo Text::_('JSAVE'); ?>
 			</button>
+			<?php if ($this->showSaveAsCopy) : ?>
+				<button type="button" class="btn btn-primary" data-submit-task="article.save2copy">
+					<span class="icon-copy" aria-hidden="true"></span>
+					<?php echo Text::_('JSAVEASCOPY'); ?>
+				</button>
+			<?php endif; ?>
 			<button type="button" class="btn btn-danger" data-submit-task="article.cancel">
 				<span class="icon-times" aria-hidden="true"></span>
 				<?php echo Text::_('JCANCEL'); ?>

language/en-GB/joomla.ini

@@ -109,6 +109,7 @@ JREGISTER="Register"
 JREQUIRED="Required"
 JRESET="Reset"
 JSAVE="Save"
+JSAVEASCOPY="Save As Copy"
 JSELECT="Select"
 JSHOW="Show"
 JSHOWPASSWORD="Show Password"

libraries/src/MVC/Controller/FormController.php

@@ -749,7 +749,7 @@ public function save($key = null, $urlVar = null)
 		{
 			case 'apply':
 				// Set the record data in the session.
-				$recordId = $model->getState($this->context . '.id');
+				$recordId = $model->getState($model->getName() . '.id');
 				$this->holdEditId($context, $recordId);
 				$app->setUserState($context . '.data', null);
 				$model->checkout($recordId);