diff --git a/game-server/app/servers/role/handler/roleHandler.ts b/game-server/app/servers/role/handler/roleHandler.ts
index da3731300..33ee83fec 100644
--- a/game-server/app/servers/role/handler/roleHandler.ts
+++ b/game-server/app/servers/role/handler/roleHandler.ts
@@ -292,7 +292,7 @@ export class RoleHandler {
 
         let { hid } = msg;
 
-        let curHero = await HeroModel.findByHidAndRole(hid, roleId, 'star colorStar quality scrollId scrollActive scrollStar scrollQuality favour favourLv');
+        let curHero = await HeroModel.findByHidAndRole(hid, roleId, 'star colorStar quality scrollId scrollActive scrollStar scrollColorStar scrollQuality favour favourLv');
         if (!curHero) return resResult(STATUS.HERO_NOT_FIND);
 
         let dicHero = gameData.hero.get(hid);
@@ -327,7 +327,7 @@ export class RoleHandler {
         } else {
             if (star > scrollStar) { // 可以升星
                 update.scrollStar++;
-            } else if (scrollQuality > quality) { // 可以升品
+            } else if (quality > scrollQuality ) { // 可以升品
                 update.scrollQuality++;
             } else if (colorStar > scrollColorStar) { // 可以升彩星
                 update.scrollColorStar++;
@@ -336,6 +336,7 @@ export class RoleHandler {
             }
 
         }
+
         let dicHeroScroll = getScollByStar(dicHero.quality, update.scrollStar, update.scrollQuality, update.scrollColorStar);
         update.scrollId = dicHeroScroll?dicHeroScroll.id: 0;