Remove @file tag docblock from all .php file

Hi Ashish.Dalvi,

Thanks for the clear explanation and patch. My CS missed that as I was missing the namespace (CS throws the error only when namespace is there as far as I can tell). I've removed the file docblock and added the namespace.

I've been having issues with the commits not getting updated in the tar/zip so please let me know if you don't see the update. It does show up in the commits stream here.

Thanks,
Scott

Unfortunately that is a breaking change. It seems the batch operations processing requires that the class be in the global namespace and therefore without a namespace tag. The @file therefore must be in place since this class cannot be namespaced (as the exception explains the @file should only be left out when namespaced).

With no @file and no namespace, Drupal CS throws an error.

Perhaps you can suggest an alternative route?

You need to add.a namespace line before the use statements. The patch to remove the comment is correct, the current code isn't.

Sorry but it's not so simple as that. If you do that and run an export, you'll see that batch processing fails as the class is no longer globally available.

The patch does not add a namespace so it remains globally available but without the namespace the @file is required.

I either need to move the callback into the module file or need to figure out how to avoid the missing class when Drupal Core batch processing tries to run the callback. I will investigate further asap.