That would be the correct (and really only) approach. Assuming that your code changes are correct and that you have recompiled the module and replaced the DotNetNuke.Modules.Documents.dll assembly in the site's /bin folder with your new version, note that you will need to

1. Clear the various search tables in the Database to remove previously indexed content that you no longer want to include.
2. Check the Documents module registration in Host --> Extensions to be certain that the Business Controller Class is correct for your revised controller if you made any changes in the module's namespace AND that the Searchable feature is still checked. If it is not checked, try updating the module definition in Host -->Extensions and re-verify that Searchable is now checked.
3. Make sure that the Search Engine Scheduler job has run (you can force it to run now in Host-->Schedule control panel) since you revised your code.

---

Bill, WESNet Designs
Team Lead - DotNetNuke Gallery Module Project (Not Actively Being Developed)
Extensions Forge Projects . . .
Current: UserExport, ContentDeJour, ePrayer, DNN NewsTicker, By Invitation
Coming Soon: FRBO-For Rent By Owner