-
Notifications
You must be signed in to change notification settings - Fork 1
Issues
is:issue state:open
is:issue state:open
Issue creation is restricted in this repository
Search results
Minor code quality improvements
code-reviewFound during code reviewFound during code reviewseverity: minorcosmetic issues or bugs with simple workaroundscosmetic issues or bugs with simple workaroundsStatus: Open.#84 In craftpulse/craft-timeloop;GQL resolvers pass raw array to service methods expecting TimeloopModel
code-reviewFound during code reviewFound during code reviewseverity: importantbugs that severely impact normal usagebugs that severely impact normal usageStatus: Open.#83 In craftpulse/craft-timeloop;GQL: inner types not registered with TypeLoader, names will collide across fields
code-reviewFound during code reviewFound during code reviewseverity: importantbugs that severely impact normal usagebugs that severely impact normal usageStatus: Open.#82 In craftpulse/craft-timeloop;Stale Craft v3 documentation link in Timeloop.php
code-reviewFound during code reviewFound during code reviewseverity: importantbugs that severely impact normal usagebugs that severely impact normal usageStatus: Open.#81 In craftpulse/craft-timeloop;ECS config targets CRAFT_CMS_4, not CRAFT_CMS_5
code-reviewFound during code reviewFound during code reviewseverity: importantbugs that severely impact normal usagebugs that severely impact normal usageStatus: Open.#80 In craftpulse/craft-timeloop;isValueEmpty() logic is inverted — field is never considered empty
code-reviewFound during code reviewFound during code reviewseverity: importantbugs that severely impact normal usagebugs that severely impact normal usageStatus: Open.#79 In craftpulse/craft-timeloop;TimeloopModel imports nystudio107\seomatic — unused, causes autoload failure without SEOmatic
code-reviewFound during code reviewFound during code reviewseverity: criticalshowstopper bugsshowstopper bugsStatus: Open.#78 In craftpulse/craft-timeloop;defineRules: commented-out rules, TODO stubs, and rules for non-existent properties
code-reviewFound during code reviewFound during code reviewseverity: importantbugs that severely impact normal usagebugs that severely impact normal usageStatus: Open.#77 In craftpulse/craft-timeloop;Timeloop.php constructor registers 'timeloop' component as plugin class, then init() overwrites
code-reviewFound during code reviewFound during code reviewseverity: importantbugs that severely impact normal usagebugs that severely impact normal usageStatus: Open.#76 In craftpulse/craft-timeloop;Missing/inconsistent section headers across multiple files
code-reviewFound during code reviewFound during code reviewseverity: importantbugs that severely impact normal usagebugs that severely impact normal usageStatus: Open.#75 In craftpulse/craft-timeloop;normalizeValue() returns empty array instead of null for empty values
code-reviewFound during code reviewFound during code reviewseverity: importantbugs that severely impact normal usagebugs that severely impact normal usageStatus: Open.#74 In craftpulse/craft-timeloop;timeloopRequired property: declared after section header, no PHPDoc, dead code
code-reviewFound during code reviewFound during code reviewseverity: importantbugs that severely impact normal usagebugs that severely impact normal usageStatus: Open.#73 In craftpulse/craft-timeloop;