Gsyfuy

I have created an event for sales_order_place_after event.

This doesn't return Order Id. However, it returns Increment ID

public function execute(Observer $observer)

    {

        $order = $observer->getEvent()->getOrder();       

        $order->getIncrementId();

        $order->getId();

    }

In Magento 2 observer I used this code, you get the order data like this,

use MagentoFrameworkEventObserverInterface;



class YOUROBSERVERCLASS implements ObserverInterface

{

    protected $orderFactory;



    public function __construct(MagentoQuoteModelQuoteFactory $quoteFactory,

    MagentoSalesModelOrder $orderFactory)

    {

        $this->orderFactory = $orderFactory;

    }



    public function execute(MagentoFrameworkEventObserver $observer)

    {

        $orderIds = $observer->getEvent()->getOrderIds();

        $lastorderId = $orderIds[0];



        $order = $this->orderFactory->load($lastorderId);

    }

}

This is an old question but given I have just faced the same problem myself I thought I would give my take on the answer. sales_order_place_after is triggered before the order is saved to db so at the point your code is running the order entity ID doesn't exist. There are few options to work around this:

You could use sales_order_save_after which is triggered after the order is actually persisted to db. This will allow you to have access to the order ID. This approach has risks however as it can be triggered again after the order is created. If you only want to run your code only once when the order is first placed, using this observer may have unexpected side effects.

Alternatively rather than using an observer you could create a plugin around MagentoSalesApiOrderManagementInterface::place. This will allow you to do what I suspect you are trying to do with sales_order_place_after.

As a rough example your modules etc/di.xml would look something like this:

<?xml version="1.0"?>

<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:ObjectManager/etc/config.xsd">

    <type name="MagentoSalesApiOrderManagementInterface">

        <plugin name="my-module-order-service" type="MyModulePluginOrderServicePlugin" sortOrder="5"/>

    </type>

</config>

And your plugin class like this:

<?php



namespace MyModulePlugin;



class OrderServicePlugin

{

    public function aroundPlace(MagentoSalesModelServiceOrderService $subject, Closure $proceed,  MagentoSalesApiDataOrderInterface $order)

    {

        $return = $proceed($order);



        $orderId = $order->getEntityId();



        // your custom code



        return $return;



    }

}

It is better to avoid around plugin unless necessary . Use after plugin or MagentoSalesApiOrderManagementInterface::place

etc/di.xml

<?xml version="1.0" ?>

    <config>

        <type name="MagentoSalesApiOrderManagementInterface">

            <plugin name="MyOrderPlaceAfterPlugin" type="MyModulePluginOrderPlaceAfterPlugin" sortOrder="99" />

        </type>

    </config>

MyModulePluginOrderPlaceAfterPlugin.php

//namespace MyModulePluginOrder;



class  PlaceAfterPlugin {





    /**

     * @param MagentoSalesApiOrderManagementInterface $orderManagementInterface

     * @param MagentoSalesModelOrderInterceptor $order

     * @return $order

     */

    public function afterPlace(MagentoSalesApiOrderManagementInterface $orderManagementInterface , $order)

    {

        $orderId = $order->getId();



        // do something with order object (Interceptor )



       return $order;

    }

}