Kdjykuj

I'm creating a cron to pass through all the products and set them visible and active.

This is my execute of the cron who runs well:

public function execute(){

    /**

     * @var $item MagentoCatalogModelProduct

     * @var MagentoCatalogModelResourceModelProductCollection $collection

     */



    $items = $this->productCollection->getItems();



    foreach($items as $item){

        echo  $item->getSku() .'<br>';

        $item->setVisibility(MagentoCatalogModelProductVisibility::VISIBILITY_BOTH);

        $item->setStatus(MagentoCatalogModelProductAttributeSourceStatus::STATUS_ENABLED);

        $item->save();

    }

    echo 'finish';

}

I've got a product with none visibility and status disabled, but when I run the function the product is still with none visibility and status disabled.

What I'm doing wrong?

You should replace the $item->save(); with below code:

$item->getResource()->saveAttribute($item, 'visibility');



$item->getResource()->saveAttribute($item, 'status');

Try to change the Collection and set the AreaCode, I've created something similar and works.

namespace NameSpaceModuleNameConsoleCommand;



use SymfonyComponentConsoleCommandCommand;

use SymfonyComponentConsoleInputInputArgument;

use SymfonyComponentConsoleInputInputInterface;

use SymfonyComponentConsoleOutputOutputInterface;



class EAVMassUpdateCommand extends Command

{

public function __construct(

       MagentoFrameworkAppState $state,

       MagentoCatalogModelResourceModelProductCollectionFactory $collectionFactory

    ) {

        $state->setAreaCode('adminhtml');

        $this->_collectionFactory = $collectionFactory;

        parent::__construct();

}



...



public function execute(){



    $items = $this->_collectionFactory->create()

        ->addAttributeToSelect('*')

        ->load();



    foreach($items as $item){

        echo  $item->getSku() .'<br>';

        $item->setVisibility(MagentoCatalogModelProductVisibility::VISIBILITY_BOTH);

        $item->setStatus(MagentoCatalogModelProductAttributeSourceStatus::STATUS_ENABLED);

        $item->save();

    }

    echo 'finish';

}

Try this way:

public function __construct(

    .....

    MagentoFrameworkAppActionContext $context

    MagentoCatalogApiProductRepositoryInterface $productRepositoryInterface,

    MagentoCatalogModelResourceModelProductCollection  $productCollection

) {

    ......

    $this->productRepository = $productRepositoryInterface;

    $this->productCollection = $productCollection;

    parent::__construct($context);

}

public function execute(){

$items = $this->productCollection->getItems();

foreach($items as $item){

    $productId =  $item->getEntityId();

    $product = $this->productRepository->getById($productId);

    $product->setStatus(MagentoCatalogModelProductAttributeSourceStatus::STATUS_DISABLED);

    $product->setVisibility(MagentoCatalogModelProductVisibility::VISIBILITY_BOTH);

    $this->productRepository->save($product);     

}

    echo 'finish';

}

For getting product collection with status enable and visibility:

you have to keep below code in block file.

protected $_coreRegistry = null;



public function __construct(

    MagentoFrameworkViewElementTemplateContext $context,

    MagentoFrameworkRegistry $registry,

    MagentoCatalogModelResourceModelProductCollectionFactory $productCollectionFactory,

    MagentoCatalogModelProductAttributeSourceStatus $productStatus,

    MagentoCatalogModelProductVisibility $productVisibility,

    array $data = 

)

{



    $this->_coreRegistry = $registry;

    $this->_productCollectionFactory = $productCollectionFactory;

    $this->productStatus = $productStatus;

    $this->productVisibility = $productVisibility;

    parent::__construct($context, $data);

}



public function getProductCollection()

{

    $collection = $this->_productCollectionFactory->create();

    $collection->addAttributeToFilter('manufacturer', ['eq' => '20']);

    $collection->addAttributeToFilter('status', ['in' => $this->productStatus->getVisibleStatusIds()]);

    $collection->setVisibility($this->productVisibility->getVisibleInSiteIds());



    return $collection;

}

Above code working for visibility type for:

• VISIBILITY_IN_SEARCH


• VISIBILITY_IN_CATALOG


• VISIBILITY_BOTH